Skip to content

Plugin rest tests for analysis-phonetic, cloud-*, and lang-* #12091

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
Merged
clintongormley merged 3 commits into from
Jul 7, 2015
Merged

Plugin rest tests for analysis-phonetic, cloud-*, and lang-* #12091

clintongormley merged 3 commits into from
Jul 7, 2015

Conversation

@clintongormley
Copy link
Contributor

@clintongormley clintongormley commented Jul 7, 2015

Closes #12062

@clintongormley clintongormley added >test Issues or PRs that are addressing/adding tests review labels Jul 7, 2015
@rmuir
Copy link
Contributor

rmuir commented Jul 7, 2015

looks great! Thank you!

clintongormley added a commit that referenced this pull request Jul 7, 2015
@clintongormley
Merge pull request #12091 from clintongormley/plugin_rest_tests
50ec1b1 
Plugin rest tests for analysis-phonetic, cloud-*, and lang-*
@clintongormley clintongormley merged commit 50ec1b1 into elastic:master Jul 7, 2015
@clintongormley clintongormley deleted the plugin_rest_tests branch July 7, 2015 17:27
@dadoonet
Copy link
Contributor

dadoonet commented Jul 7, 2015

I have some issues while running Rest tests in phonetic module.
I did not test other modules yet.

mvn clean install -DskipTests ; cd plugins/analysis-phonetic; mvn install -Dtests.rest=true

gives

Tests run: 4, Failures: 2, Errors: 0, Skipped: 0, Time elapsed: 8.213 sec <<< FAILURE! - in org.elasticsearch.index.analysis.AnalysisPhoneticRestIT
test {yaml=analysis_phonetic/10_metaphone/Metaphone}(org.elasticsearch.index.analysis.AnalysisPhoneticRestIT)  Time elapsed: 4.397 sec  <<< FAILURE!
java.lang.AssertionError: expected [2xx] status code but api [indices.analyze] returned [503 Service Unavailable] [{"error":{"root_cause":[{"type":"no_shard_available_action_exception","reason":"No shard available for [org.elasticsearch.action.admin.indices.analyze.AnalyzeRequest@26dd825d]"}],"type":"no_shard_available_action_exception","reason":"No shard available for [org.elasticsearch.action.admin.indices.analyze.AnalyzeRequest@26dd825d]"},"status":503}]
        at __randomizedtesting.SeedInfo.seed([FBEBE5457042DE03:73BFDA9FDEBEB3FB]:0)
        at org.junit.Assert.fail(Assert.java:88)
        at org.elasticsearch.test.rest.section.DoSection.execute(DoSection.java:99)
        at org.elasticsearch.test.rest.ElasticsearchRestTestCase.test(ElasticsearchRestTestCase.java:373)
        at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
        at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:57)
        at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
        at java.lang.reflect.Method.invoke(Method.java:606)
        at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1627)
        at com.carrotsearch.randomizedtesting.RandomizedRunner$6.evaluate(RandomizedRunner.java:836)
        at com.carrotsearch.randomizedtesting.RandomizedRunner$7.evaluate(RandomizedRunner.java:872)
        at com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:886)
        at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at org.apache.lucene.util.TestRuleSetupTeardownChained$1.evaluate(TestRuleSetupTeardownChained.java:50)
        at org.apache.lucene.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:46)
        at org.apache.lucene.util.TestRuleThreadAndTestName$1.evaluate(TestRuleThreadAndTestName.java:49)
        at org.apache.lucene.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:65)
        at org.apache.lucene.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:48)
        at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:365)
        at com.carrotsearch.randomizedtesting.ThreadLeakControl.forkTimeoutingTask(ThreadLeakControl.java:798)
        at com.carrotsearch.randomizedtesting.ThreadLeakControl$3.evaluate(ThreadLeakControl.java:458)
        at com.carrotsearch.randomizedtesting.RandomizedRunner.runSingleTest(RandomizedRunner.java:845)
        at com.carrotsearch.randomizedtesting.RandomizedRunner$3.evaluate(RandomizedRunner.java:747)
        at com.carrotsearch.randomizedtesting.RandomizedRunner$4.evaluate(RandomizedRunner.java:781)
        at com.carrotsearch.randomizedtesting.RandomizedRunner$5.evaluate(RandomizedRunner.java:792)
        at org.apache.lucene.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:46)
        at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at org.apache.lucene.util.TestRuleStoreClassName$1.evaluate(TestRuleStoreClassName.java:42)
        at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:39)
        at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:39)
        at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at org.apache.lucene.util.TestRuleAssertionsRequired$1.evaluate(TestRuleAssertionsRequired.java:54)
        at org.apache.lucene.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:48)
        at org.apache.lucene.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:65)
        at org.apache.lucene.util.TestRuleIgnoreTestSuites$1.evaluate(TestRuleIgnoreTestSuites.java:55)
        at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
        at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:365)
        at java.lang.Thread.run(Thread.java:745)

test {yaml=analysis_phonetic/40_search/Index phonetic content}(org.elasticsearch.index.analysis.AnalysisPhoneticRestIT)  Time elapsed: 1.138 sec  <<< FAILURE!
java.lang.AssertionError: field [hits.total] doesn't match the expected value
Expected: <1>
     but: was <0>

Do you guys have the same kind of issue? no_shard_available_action_exception

@dadoonet dadoonet mentioned this pull request Jul 7, 2015
Closed
@dadoonet
Copy link
Contributor

dadoonet commented Jul 7, 2015

For the record, opened issue for this #12099

dadoonet added a commit that referenced this pull request Jul 8, 2015
@dadoonet
[phonetic] move integration tests to REST tests
b970fba 
We can keep only unit tests in plugins instead of starting each time a local node and running tests against it.

Also follow up of #12091
@clintongormley clintongormley mentioned this pull request Jul 8, 2015
Closed
@dadoonet dadoonet mentioned this pull request Jul 8, 2015
Merged
dadoonet added a commit to dadoonet/elasticsearch that referenced this pull request Jul 8, 2015
@dadoonet
[ICU] move integration tests to REST tests
4738fa0 
We can keep only unit tests in plugins instead of starting each time a local node and running tests against it.

Also follow up of elastic#12091

Tests can use more than one JVM

We don't need anymore to set the number of jvm to run tests as we moved IT to Rest Tests
For all plugins but cloud plugins which will require another way for running integration tests.
dadoonet added a commit to dadoonet/elasticsearch that referenced this pull request Jul 8, 2015
@dadoonet
[Smartcn] move integration tests to REST tests
7a1255d 
We can keep only unit tests in plugins instead of starting each time a local node and running tests against it.

Also follow up of elastic#12091
dadoonet added a commit to dadoonet/elasticsearch that referenced this pull request Jul 8, 2015
@dadoonet
[Stempel] move integration tests to REST tests
19b2e77 
We can keep only unit tests in plugins instead of starting each time a local node and running tests against it.

Also follow up of elastic#12091
dadoonet added a commit to dadoonet/elasticsearch that referenced this pull request Jul 8, 2015
@dadoonet
[Javascript] move integration tests to REST tests
c99bc81 
We can keep only unit tests in plugins instead of starting each time a local node and running tests against it.

Also follow up of elastic#12091
dadoonet added a commit to dadoonet/elasticsearch that referenced this pull request Jul 8, 2015
@dadoonet
[python] move integration tests to REST tests
97019ce 
We can keep only unit tests in plugins instead of starting each time a local node and running tests against it.

Also follow up of elastic#12091
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

>test Issues or PRs that are addressing/adding tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@clintongormley @rmuir @dadoonet