in run_tests.py, exception thrown by 'git remote add origin' when the remote already exists will not exit
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* updated files with docstrings to pass pylint
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* updated samples to prepare for enabling missing-docstring linter; will continue to work on this before committing setup.cfg
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* removed missing-function-docstring from setup.cfg so the linter doesn't fail while work on docstrings continues
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* corrected unnecessary return docstring values
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* fixing failure in 'black' on reformatting
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* updated utils to pass missing-function-docstring tests
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* updated functions with missing docstrings or pylint ignore instructions; added a utility to automatically add these ignore instructions to most functions that should be self-describing; rolled back some automatically generated code mistakenly changed
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* * ignoring opensearchpy for pylint and then added it back to noxfile.py
* fixed some lints; created a feature flag for newer dynamic pylint so now lints can be fixed first in legacy code and then enabled by multiple people
* extracted a method for per-folder linting
* updated noxfile.lint_per_folder with type hints
* enabled unspecified-encoding in pylint
* added disable missing-function-docstring pragma to test_clients.py in test_async and test_server
* added more encodings to pass unspecified-encoding pylint tests
* updated changelog
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* updated CHANGELOG.md entry
removed the feature flag for pylint lint_per_folder
fixed failures from mypy and pylint
removed pylint MESSAGE CONTROL config from setup.cfg after relocating to lint_per_folder method
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* removed pylint ignore missing-function-docstring
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* added pylint.extensions.docparams plugin
updated some docstrings to correct parameters
removed pylint from setup.cfg
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* added four lints for opensearchpy/
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* adding await back to client.info() call
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* updated TODOs as requested
renamed test_opensearchpy.test_async.test_server.test_helpers.conftest.setup_ubq_tests to setup_update_by_query_tests
added
OpenSearch-main/rest-api-spec/src/main/resources/rest-api-spec/test/indices/stats/50_noop_update[0]
to skip tests list
run_tests.py catches a CalledProcessError when the git repo already exists and the command to add the origin fails in fetch_opensearch_repo()
Signed-off-by: Mark Cohen <markcoh@amazon.com>
---------
Signed-off-by: Mark Cohen <markcoh@amazon.com>
* feat(plugins): add index management client plugin
Signed-off-by: florian <florian@harfanglab.fr>
* chore(CHANGELOG): added entry for ISM api support in changelog
Signed-off-by: florian <florian@harfanglab.fr>
* test(plugins): use assertEqual to compare call parameters
Signed-off-by: florian <florian@harfanglab.fr>
* test(plugins): edit policy to support older versions of opensearch
Signed-off-by: florian <florian@harfanglab.fr>
* test(plugins): ignore plugin tests when opensearch is unreleased
Signed-off-by: florian <florian@harfanglab.fr>
* test(plugins): move plugin tests into separate files
Signed-off-by: florian <florian@harfanglab.fr>
* test(plugins): fix import of OpenSearchTestCase
Signed-off-by: florian <florian@harfanglab.fr>
* chore(USER_GUIDE): add a index management plugin part
Signed-off-by: florian <florian@harfanglab.fr>
---------
Signed-off-by: florian <florian@harfanglab.fr>
* [#387] Do not escape the "timeout" parameter.
When the "timeout" parameter is escaped and turned into
a string, it does not pass validation checks.
This PR ensures that timeout is passed through as-is,
just like "request_timeout".
There's no explicit test for this parameter, but I included
it on some existing tests just to ensure that the code
is exercised. Without the fix in place, one of the tests as modified will not pass.
Signed-off-by: Charles Greer <cgreer@lexmachina.com>
* Unit test for timeout parameter
Revert original test modifications.
Signed-off-by: Charles Greer <cgreer@lexmachina.com>
* Extend test and linter
Signed-off-by: Charles Greer <cgreer@lexmachina.com>
---------
Signed-off-by: Charles Greer <cgreer@lexmachina.com>
* Breaking changes for type removal
Signed-off-by: Vacha Shah <vachshah@amazon.com>
* Adding support to test against unreleased OpenSearch
Signed-off-by: Vacha Shah <vachshah@amazon.com>
* Formatting
Signed-off-by: Vacha Shah <vachshah@amazon.com>
* Addressing comments to refactor run-opensearch.sh
Signed-off-by: Vacha Shah <vachshah@amazon.com>
AbitraryYu