Π“Ρ€Π°Ρ„ ΠΊΠΎΠΌΠΌΠΈΡ‚ΠΎΠ²

3 ΠšΠΎΠΌΠΌΠΈΡ‚ΠΎΠ²

Автор SHA1 Π‘ΠΎΠΎΠ±Ρ‰Π΅Π½ΠΈΠ΅ Π”Π°Ρ‚Π°
Hannah K 743fd0893c
Hotfix: Progressbar/Time Remaining More Accurate During Import (#11)
* Swaps in the progress bar update unit to the row and not the document
size. Also adds in more references to the delimiter and file endings
variables for easier future updating,

* Fixes a possible capitalization problem with a linux build.

* Adds the flag in `pip install` to not use PEP517 per a discussion on
https://github.com/pypa/pip/issues/6163

* Removes the `--no-use-pep517` flag on the Mac build. Prematurely added.
2019-02-05 13:54:55 -05:00
Hannah K b69e11596d
Creates the `cadet import` command (#4)
* Creates the starting point for the CLI tool.

* Aligns the code to the Specs.

* Adds a number of try-catch statements for better error-handling.

* Removes a comment that was wrong (the raised error is hit).

* Adds a docstring to the beginning of the file.

* Updates the flags/options to match the portal's vocabulary, and
corresponding variable names. Swaps out print() and puts in
click.echo(). Adds in a try/catch for the document upsert

* Changes how a click.echo message was formatted to use correct syntax.

* Unifies which string quotation marks are used (now single instead of a
mix); Adds new lines in exception messages for readability; Moves the
source_size variable check inside of a try/catch so that if the source
file provided doesn't exist, it fails gracefully.

* Adds a check for the source file's extension being CSV or TSV and
raising an exception if it's not.

* Moves a comment and removes a bunch of blank lines.

* Feature: Add CI and release pipeline (#3)

* add azure-pipelines.yml

* update?

* add test step

* drop travis

* add badge

* update contrib

* pr validation, package on master

* move condition for github

* added a linting step via pylint before test

* updated so that all stages lint and that the correct module is passed into linter

* Removes the newline characters from the messages since it's clunky.

* Adds a `required=True` on the collection name, database name, and type
flags so that if any of them are missing, then it should fail. The
connection flags have separate fail-safes in place so they don't need
the `required=True`.

* Pulls out some functionality into separate functions for easier testing;
adds function descriptions for those new functions; fixes a connection
typo.

* Corrects unnecessary white space at the end of a few lines.

* Cleans up from pylint suggestions.

* Updates a function comment

* Add initial tests and test files (#5)

* Add initial tests and test files

* Add newline

* Import mock

* Add test import

* Add testing, remove test cases that rely on auth response

* Add connection string parsing and test

* Remove extra line, and unused import statement

* Modify tests and test file

* Remove optionality of arguments

* Add comments on test

* Mock on upsertedDocs

* Rewrite test mocking

* Remove print statement

* Add comment to describe test

* Fix spacing, typo

* Creates the starting point for the CLI tool.

* Aligns the code to the Specs.

* Adds a number of try-catch statements for better error-handling.

* Removes a comment that was wrong (the raised error is hit).

* Updates the flags/options to match the portal's vocabulary, and
corresponding variable names. Swaps out print() and puts in
click.echo(). Adds in a try/catch for the document upsert

* Changes how a click.echo message was formatted to use correct syntax.

* Adds a docstring to the beginning of the file.

* Unifies which string quotation marks are used (now single instead of a
mix); Adds new lines in exception messages for readability; Moves the
source_size variable check inside of a try/catch so that if the source
file provided doesn't exist, it fails gracefully.

* Adds a check for the source file's extension being CSV or TSV and
raising an exception if it's not.

* Moves a comment and removes a bunch of blank lines.

* Removes the newline characters from the messages since it's clunky.

* Adds a `required=True` on the collection name, database name, and type
flags so that if any of them are missing, then it should fail. The
connection flags have separate fail-safes in place so they don't need
the `required=True`.

* Pulls out some functionality into separate functions for easier testing;
adds function descriptions for those new functions; fixes a connection
typo.

* Corrects unnecessary white space at the end of a few lines.

* Cleans up from pylint suggestions.

* Updates a function comment

* Add initial tests and test files (#5)

* Add initial tests and test files

* Add newline

* Import mock

* Add test import

* Add testing, remove test cases that rely on auth response

* Add connection string parsing and test

* Remove extra line, and unused import statement

* Modify tests and test file

* Remove optionality of arguments

* Add comments on test

* Mock on upsertedDocs

* Rewrite test mocking

* Remove print statement

* Add comment to describe test

* Fix spacing, typo

* Moves things from my previous branch into the style from master. Updates
the CONTRIBUTING and README docs.

* The first(-ish) pass of the linting/reorganizing.

* Updates some dependencies.

* Updates the Azure pipelines to run `pip3` since we require Python 3.

* Changes to pipeline to see if pylint and pytest.

* Separates the installing of required modules from the actual testing. Is
there a synchronous issue?

* Makes the change to the Windows build since that is where the failure is
happening.

* Removes a broken requirement that was possibly introduced.

* Feature/linting tests (#6)

* Add docstrings, remove instances of 'self' on classes
* Fix file importation and reading
* Re-add 'connection-string'
* Add conditional, to allow users to type in source_path
* Change variable names, edit tests
* Consistency on click options

* Feature/add description to pylint ignore (#7)

* Fix typo

* Add descriptive pylint ignore message

* Update test_cadet.py

* Updates the contributing docs to include a note about a known bug.

* Splits up the installing and testing for the linux build..
2019-01-29 15:21:42 -05:00
Ben Greenier 10d77b7546
Feature: Add CI and release pipeline (#3)
* add azure-pipelines.yml

* update?

* add test step

* drop travis

* add badge

* update contrib

* pr validation, package on master

* move condition for github

* added a linting step via pylint before test

* updated so that all stages lint and that the correct module is passed into linter
2019-01-10 10:17:37 -05:00