Code

New test framework#1967

Code

Merged

peternied merged 47 commits intoopensearch-project:mainopensearch-project/security:mainfrom eliatra:integration-test-frameworkeliatra/OpenSearch-security-tests:integration-test-frameworkCopy head branch name to clipboardSep 20, 2022

+3,632-31

Commits

Commits on Sep 13, 2022

• New test framework

  Show description for 1efe734
  

  nibix

  authored and

  lukasz-soszynski-eliatra

  committed
  1efe734

  Copy full SHA for 1efe734

• removed abstract base class, prefer composition over inheritance

  Show description for c3ae9a6
  

  jochenkressin

  authored and

  lukasz-soszynski-eliatra

  committed
  c3ae9a6

  Copy full SHA for c3ae9a6

• moved Jackson dependencies from test classes to TestRestClient

  Show description for 6db0e64
  

  jochenkressin

  authored and

  lukasz-soszynski-eliatra

  committed
  6db0e64

  Copy full SHA for 6db0e64

• removed unused code, replaced some old ES references

  Show description for 1fa6c0a
  

  jochenkressin

  authored and

  lukasz-soszynski-eliatra

  committed
  1fa6c0a

  Copy full SHA for 1fa6c0a

• replaced JUnit asserts with Hamcrest asserts

  Show description for 895e396
  

  jochenkressin

  authored and

  lukasz-soszynski-eliatra

  committed
  895e396

  Copy full SHA for 895e396

• renamed ClusterConfiguration to ClusterManager

  Show description for b4cf9e9
  

  jochenkressin

  authored and

  lukasz-soszynski-eliatra

  committed
  b4cf9e9

  Copy full SHA for b4cf9e9

• throw exception if index already exists

  Show description for c79c68e
  

  jochenkressin

  authored and

  lukasz-soszynski-eliatra

  committed
  c79c68e

  Copy full SHA for c79c68e

• fixed incorrect test

  Show description for 8f94acf
  

  jochenkressin

  authored and

  lukasz-soszynski-eliatra

  committed
  8f94acf

  Copy full SHA for 8f94acf

• fixed code hygiene issues

  Show description for f1cef31
  

  jochenkressin

  authored and

  lukasz-soszynski-eliatra

  committed
  f1cef31

  Copy full SHA for f1cef31

• TestRestClient uses unchecked exceptions

  Show description for 6f4dae9
  lukasz-soszynski-eliatra

  committed
  6f4dae9

  Copy full SHA for 6f4dae9

• List of plugins in ClusterManager is immutable.

  Show description for bbf5dc9
  lukasz-soszynski-eliatra

  committed
  bbf5dc9

  Copy full SHA for bbf5dc9

• Unused and overloaded version of user method removed from LocalCluste cluss.

  Show description for 0289749
  lukasz-soszynski-eliatra

  committed
  0289749

  Copy full SHA for 0289749

• Unused constructor removed from class ContextHeaderDecoratorClient

  Show description for f81ccc4
  lukasz-soszynski-eliatra

  committed
  f81ccc4

  Copy full SHA for f81ccc4

• fix build.gradle - new test configuration, dependencies, desable forbiddenApisNewTest

  Show description for 3e6179f
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  3e6179f

  Copy full SHA for 3e6179f

• Try with resources was used to close OpenSearch clinet in the LocalCluster class.

  Show description for f373131
  lukasz-soszynski-eliatra

  committed
  f373131

  Copy full SHA for f373131

• TestCertificates - throw RuntimeException instead of IOException

  Show description for d135827
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  d135827

  Copy full SHA for d135827

• Nodes objects are stored in one list (instead of four) in class LocalOpenSearchCluster. Methods which return random node removed.

  Show description for 69dfa18
  lukasz-soszynski-eliatra

  committed
  69dfa18

  Copy full SHA for 69dfa18

• OpenSearch claster is stopped immediatly, without invocations of thread sleep.

  Show description for 9c1a35d
  lukasz-soszynski-eliatra

  committed
  9c1a35d

  Copy full SHA for 9c1a35d

• rename newTest to integrationTest

  Show description for 26f098b
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  26f098b

  Copy full SHA for 26f098b

• Method org.opensearch.test.framework.cluster.LocalOpenSearchCluster.Node#start returns StartStage enum instead of string.

  Show description for 69e3d31
  lukasz-soszynski-eliatra

  committed
  69e3d31

  Copy full SHA for 69e3d31

• Method org.opensearch.test.framework.cluster.LocalCluster.Builder#nodeSettings accepts map instead of varargs.

  Show description for 216153e
  lukasz-soszynski-eliatra

  committed
  216153e

  Copy full SHA for 216153e

• Method getUnitTestForkNumber removed, system property forkno is no longer used.

  Show description for 85f599f
  lukasz-soszynski-eliatra

  committed
  85f599f

  Copy full SHA for 85f599f

• ClusterBlockException does not appear in the course of integration tests. Log4j2 reads configuration from src/integrationTest/resources/log4j2-test.properties

  Show description for c32984e
  lukasz-soszynski-eliatra

  committed
  c32984e

  Copy full SHA for c32984e

• Unused code related to flag trackResources removed from class TestRestClient.

  Show description for 5b22c51
  lukasz-soszynski-eliatra

  committed
  5b22c51

  Copy full SHA for 5b22c51

• Redundant code in method org.opensearch.test.framework.cluster.LocalCluster#after replaced by close() invocation

  Show description for 075adfa
  lukasz-soszynski-eliatra

  committed
  075adfa

  Copy full SHA for 075adfa

• Rename from ClusterConfiguration to ClusterManager finished.

  Show description for f217c35
  lukasz-soszynski-eliatra

  committed
  f217c35

  Copy full SHA for f217c35

• Correction in class MinimumSecuritySettingsSupplierFactory to pass checkstyle tests, licence header added where missing, test classes moved to proper packages.

  Show description for 65df948
  lukasz-soszynski-eliatra

  committed
  65df948

  Copy full SHA for 65df948

• Netty dependencies conflict resolved

  Show description for ea4251c
  lukasz-soszynski-eliatra

  committed
  ea4251c

  Copy full SHA for ea4251c

• change version of commons-io:commons-io for test and integrationTest

  Show description for 237a49b
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  237a49b

  Copy full SHA for 237a49b

• Word master replaced with cluster manager in method and local variables names

  Show description for c0b23d2
  lukasz-soszynski-eliatra

  committed
  c0b23d2

  Copy full SHA for c0b23d2

• Method LocalOpenSearchCluster#stop simplified.

  Show description for 9f6a60c
  lukasz-soszynski-eliatra

  committed
  9f6a60c

  Copy full SHA for 9f6a60c

• integrationTest - upgrade org.hamcrest:hamcrest to 2.2

  Show description for a418b64
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  a418b64

  Copy full SHA for a418b64

• add tests for SocketUtils class

  Show description for 5340d9a
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  5340d9a

  Copy full SHA for 5340d9a

• add new line to SocketUtilsTests class

  Show description for e2c66c9
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  e2c66c9

  Copy full SHA for e2c66c9

• add separate spotless configuration for integrationTest

  Show description for 777096e
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  777096e

  Copy full SHA for 777096e

• integrationTest - apply spotless formatting

  Show description for 652af5b
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  652af5b

  Copy full SHA for 652af5b

• CI/CD - run integrationTest task only when test task is executed

  Show description for df0da3c
  

  kt-eliatra

  authored and

  lukasz-soszynski-eliatra

  committed
  df0da3c

  Copy full SHA for df0da3c

• Removed uses of NestedValueMap and replaced by ToXContent

  Show description for 9f4d715
  

  nibix

  authored and

  lukasz-soszynski-eliatra

  committed
  9f4d715

  Copy full SHA for 9f4d715

• Fixed imports

  Show description for 33f35f5
  

  nibix

  authored and

  lukasz-soszynski-eliatra

  committed
  33f35f5

  Copy full SHA for 33f35f5

• Modify defaults to prevent warnings from being added to the test output

  Show description for f1bce59
  

  peternied

  authored and

  lukasz-soszynski-eliatra

  committed
  f1bce59

  Copy full SHA for f1bce59

• Replace non-inclusive terms from configuration settings

  Show description for 33b78b3
  

  peternied

  authored and

  lukasz-soszynski-eliatra

  committed
  33b78b3

  Copy full SHA for 33b78b3

• Output of integration tests cleaned.

  Show description for 674be40
  lukasz-soszynski-eliatra

  committed
  674be40

  Copy full SHA for 674be40

• Added API docs

  Show description for 0757c08
  

  nibix

  authored and

  lukasz-soszynski-eliatra

  committed
  0757c08

  Copy full SHA for 0757c08

• Applied spotless rules

  Show description for 08d6500
  

  nibix

  authored and

  lukasz-soszynski-eliatra

  committed
  08d6500

  Copy full SHA for 08d6500

• A temporary solution to limit the number of warning messages caused by class NodeAndClusterIdConverter during tests execution

  Show description for 67b5446
  lukasz-soszynski-eliatra

  committed
  67b5446

  Copy full SHA for 67b5446

• Fixed license headers

  Show description for 6e5e885
  nibix

  committed
  6e5e885

  Copy full SHA for 6e5e885

Commits on Sep 19, 2022

• Correction after code review, removed some comments and one javadoc added.

  Show description for 9fdb73a
  lukasz-soszynski-eliatra

  committed
  9fdb73a

  Copy full SHA for 9fdb73a