Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add test suite #1

Merged
merged 141 commits into from
May 17, 2024
Merged

Add test suite #1

merged 141 commits into from
May 17, 2024

Conversation

claucambra
Copy link
Owner

No description provided.

@claucambra claucambra self-assigned this May 17, 2024
@claucambra
Add RemoteInterface protocol
b5d93ac
@claucambra
Add RemoteRequestOptions struct
e706dae
@claucambra
Add convenience extension to convert request options to NKRequestOptions
9167b8d
@claucambra
Add RemoteFileMetadata protocol
0133a7a
@claucambra
Add conformance to RemoteFileMetadata in NKFile
59cfc0d
@claucambra
Add abstracted types in RemoteInterface
8b55683
@claucambra
Add conformance to RemoteInterface in NextcloudKit
0dc07c3
@claucambra
Remove unused NextcloudKit import in files database manager
ae3561a
@claucambra
Add remote error type enum
0a92611
@claucambra
Use remote error type enum in NKError extension
1dd3285
@claucambra
Use NSFileProviderError rather than default Error in RemoteInterface
13ce06c
@claucambra
Fix error code in enumerator sync engine
85d5589
@claucambra
Use await/async for enumerator
71dcb67
@claucambra
Stop with the unnecessary abstractions
17b0704 
This reverts commit 04cbc1f.

Revert "Add conformance to RemoteFileMetadata in NKFile"

This reverts commit e15869b.

Revert "Add RemoteFileMetadata protocol"

This reverts commit 33c9d4f.

Revert "Add convenience extension to convert request options to NKRequestOptions"

This reverts commit 97fec94.

Revert "Add RemoteRequestOptions struct"

This reverts commit 9722a35.

Revert "Use remote error type enum in NKError extension"

This reverts commit 2e86d42.

Revert "Add remote error type enum"

This reverts commit d93ed04.

Kill off non NK types and errors and rely on the core library, the abstractions are not providing value
@claucambra
Use RemoteInterface in Enumerator SyncEngine extension
8ab603e
@claucambra
Add move method to RemoteInterface protocol
0347266
@claucambra
Remove getInternalType method from RemoteInterface protocol
482ac04
@claucambra
Use RemoteInterface in items
3644a83
@claucambra
Use RemoteInterface in Enumerator
4b2c8be
@claucambra
Use RemoteInterface in item creation
0cddb23
@claucambra
Add fetchCapabilities to RemoteInterface
f2a4de2
@claucambra
Add setDelegate to RemoteInterface
d978c63
@claucambra
Use RemoteInterface in RemoteChangeObserver
4d6a270
@claucambra
Remove unused getInternalType method
f6e2b5c
@claucambra
Register capabilities fetch task
c195048
@claucambra
Add basic MockRemoteItem
292e9fd
@claucambra
Add placeholder starter MockRemoteInterface
c857c5d 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Make MockRemoteItem equatable
aed551b 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add method to fetch item in MockRemoteInterface within node tree
f1c9b31 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add a basic test for remote change observer
a6cc100 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Make some members of RemoteChangeObserver more public
d452ff1 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add ability to inject delay in mock notify push server response
40298d9 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add reset method in mocknotifypushserver
9ed60a4 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add test for authentication retry with remotechangeobserver
03a4dc7 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Increase interval between pings for remote change observer
de068f8 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add delay before reconfiguration attempts for remote change observer
8719a0d 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Make timeouts and failure counts configurable in remote change observer
d3be053 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add property indicating if websockettask is active
bff9ba0 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add some logging to mock notify push server
1455563 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Test that the remote change observer stops trying after limit
3cfd4c9 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add ability to send message through to connected clients in mocknotif…
3a45a6f 
…ypushserver

Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Test that remote change observer properly notifies changes
990e992 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add test to ensure non file notifications are ignored by remote chang…
1b984fd 
…e observer
@claucambra
Add property to remote change observer to check if polling is active
0e67230
@claucambra
Allow setting of poll interval for remote change observer
b4d3cc7 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Improve polling test for remote change observer
bcc1148 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Allow resetting of credentials independently of other variables in mo…
74351c1 
…ck notify push server

Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add ability to close connections in mock notify push server
d4bf581 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Do not trigger double reconnect attempts in remote change observer wh…
45ce5ae 
…en connection closed

Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add method to manually trigger connection attempt in remote change ob…
7cee706 
…server

Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add test for remote change observer reconnecting on socket close
4c6c541 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add tests for account
3ea44da 
Signed-off-by: Claudio Cambra <[email protected]>
@claucambra
Add ability to refuse conection to MockNotifyPushServer
679dbf7 
Signed-off-by: Claudio Cambra <[email protected]>
@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@claucambra claucambra merged commit fa8bee3 into main May 17, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@claucambra claucambra

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@claucambra @codecov-commenter