-
Notifications
You must be signed in to change notification settings - Fork 5
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update appinfo and fix User backend OC_User_* not found #12
Conversation
This gets rid of errors in the "unit tests" where I am getting CI going in another branch.
Now I get:
which is "better" - now the "unit test" issue is just because the tests are currently relying on the various back-ends being available. |
d04d718
to
1c68484
Compare
@phil-davis These are NOT unit tests - they depend on FTP/SMB server and won't pass until the env has the servers |
I realise that - they are close to being acceptance tests, since they need real external services. So someone needs to decide how we proceed here, e.g.
|
1aac65b
to
2b77871
Compare
2b77871
to
c5f3271
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is "a good thing".
The state of the "unit" tests that are in this app is a different issue that needs to be sorted out separately from the fix/workaround here.
Fixes #9