-
Notifications
You must be signed in to change notification settings - Fork 74
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
phpunit tests converted to mocha/chai #140
Open
abaevbog
wants to merge
33
commits into
zotero:master
Choose a base branch
from
abaevbog:mocha-chai-tests
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 1 commit
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
cfc8228
phpunit test v1 and v2 converted to mocha/chai
abaevbog d58206e
helper functions for api v3, annotation, atom and creator tests for v3
abaevbog 0244f16
v3 collection, group, fullText and note test
abaevbog f4dfc10
v3 object test
abaevbog b30737f
leftover object tests, mappings, params tests
abaevbog aae33b1
previously skipped tests requiring citation or translation server
abaevbog 878afef
file tests v2 and half of v3
abaevbog e8449db
the rest of file tests
abaevbog 8b7804b
publication tests
abaevbog bf083e1
minor adjustments to prevent random 500 errors
abaevbog 6af7367
more v3 tests
abaevbog 9e746e0
removing work dir, removed unneeded data/ files
abaevbog 339d923
notification, tag and version tests
abaevbog 2f5ccd0
v3 itemTest
abaevbog 755e133
some cleanup, proper retry logic when socket hangs
abaevbog 319ce22
using node-config
abaevbog 1ee44b1
removed inheritance from between api files and helpers
abaevbog 41d5981
package.json
abaevbog 4d7632c
changed 'Setup' for 'Before' and 'Wrapup' for 'After' in shared.js, g…
abaevbog e150cd1
removing retries, using 0-second delay instead
abaevbog ae270e5
minor cleanup, added missing comments, some tweaking with socker hanging
abaevbog 0d653ed
using helper functions for statusForObject + removed unnecessary part…
abaevbog 5e21351
initial pdf text extraction setup
abaevbog 8e0936b
pdf text extraction test
abaevbog 1976445
full-text tests with indexing function invoked locally
abaevbog d8f3427
items/unfiled and items/unfiled/tags tests
abaevbog 2e7a853
?sort=editBy for group libraries
abaevbog d4465f2
test to rename tag for PR #150
abaevbog e7518c2
test for literal || escaping for tags PR #143
abaevbog c5b1a93
new annotation types test
abaevbog 640caf4
minor cleanup, skip tests for PRs, missing tests
abaevbog 8253f4d
skip annotation tags test for collections
abaevbog b2b562a
should_add_zero_integer_value_for_lastPageIndex
abaevbog File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
You should add a comment with a link to the GitHub issue so we can track that bug.
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.
I just meant to comment the line so we know why this is here, not to add a comment on GitHub.
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.
Got it! My bad
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.
I mean literally add a comment to the source code above this line and commit it. Nothing to do with our GitHub. There just shouldn’t be random unexplained lines in the source code that will stay there forever because we don’t know why they’re there.
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.
Understood. Sorry about that