Comments (2)
@phil-davis, currently, if the federation
app cannot be started, there are no imported modules that we can use in the testing
app. This would have been fine if the Http response to the APIs in the testing
app would throw some 4XX/5XX status code (better response is always better, though).
Sadly, the response is 200 OK
and empty body content, which will pass most of our assertions (where we check that status code is 200
) and gives very ambiguous parsing errors (eg. where we do res.json()
).
When the testing app fails, it should give 5XX errors, not 200
. We should prioritize this at least in this sprint or the very next.
Try yourself:
- Disable
federation
curl http://admin:admin@localhost/core/ocs/v1.php/apps/testing/api/v1/file --verbose
from testing.
This was noticed when trying to run a selection of acceptance tests against a system that had federation
disabled. In that situation we should still be able to run "ordinary" acceptance test scenarios that do not need or test the federation features.
I added this to "Backlog for planning" so that it gets noticed, and someone can at least have a short look and report what might be possible to do.
from testing.
Related Issues (20)
- Add new routes for handling trusted servers in owncloud
- Fix failing notifications test HOT 1
- CI failing on trusted server tests
- Acceptance test fails due to core changes
- UniqueConstraintViolationException inside log when adding trusted servers using testing api
- Undefined index: value HOT 2
- Acceptance tests failing after core changes
- Add php-cs-fixer HOT 1
- increaseFileIDsBeyondMax32bits triggers "ORA-00942: table or view does not exist"
- Add getAppValue and getAppValues functionality HOT 1
- occ commands with spaces in quoted strings are not parsed correctly HOT 1
- Why do we escapeshellarg the parts of occ commands? HOT 1
- Add more test coverage for the testing app HOT 7
- Ability to specify OCC command HOT 1
- Remove steps moved to core
- Fix steps changed in core
- Fix steps changed in core HOT 1
- add an occ command to mass-create users
- Create cli acceptance tests
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from testing.