Skip to content
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

Reenable import objects tests #21250

Merged
merged 2 commits into from
Jul 27, 2018

Conversation

cjcenizal
Copy link
Contributor

Added primary callout to indicate completion of import process when the user has opted to not import anything.

image

@elasticmachine
Copy link
Contributor

💔 Build Failed

@cjcenizal
Copy link
Contributor Author

cjcenizal commented Jul 25, 2018

Intake failure:

22:26:06   1) dev/build/lib/fs copyAll() copies files and directories from source to dest, creating dest if necessary, respecting mode:
22:26:06      Error: expected '755' to equal '777'
22:26:06       at Assertion.assert (node_modules/expect.js/index.js:96:13)
22:26:06       at Assertion.be.Assertion.equal (node_modules/expect.js/index.js:216:10)
22:26:06       at Assertion.(anonymous function) [as be] (node_modules/expect.js/index.js:69:24)
22:26:06       at Context.it (src/dev/build/lib/__tests__/fs.js:228:78)
22:26:06       at <anonymous>

X-Pack failure:

22:21:09   1) ML - validateJob categorization job using mlcategory passes aggregatable field check:
22:21:09 
22:21:09       Error: expected [ 'job_id_valid',
22:21:09   'detectors_function_not_empty',
22:21:09   'index_fields_valid',
22:21:09   'field_not_aggregatable',
22:21:09   'time_field_invalid' ] to sort of equal [ 'job_id_valid',
22:21:09   'detectors_function_not_empty',
22:21:09   'index_fields_valid',
22:21:09   'success_cardinality',
22:21:09   'time_field_invalid',
22:21:09   'influencer_low_suggestion' ]
22:21:09       + expected - actual
22:21:09 
22:21:09        [
22:21:09          "job_id_valid"
22:21:09          "detectors_function_not_empty"
22:21:09          "index_fields_valid"
22:21:09       -  "field_not_aggregatable"
22:21:09       +  "success_cardinality"
22:21:09          "time_field_invalid"
22:21:09       +  "influencer_low_suggestion"
22:21:09        ]
22:21:09       
22:21:09       at Assertion.assert (node_modules/expect.js/index.js:96:13)
22:21:09       at Assertion.eql (node_modules/expect.js/index.js:230:10)
22:21:09       at then.messages (plugins/ml/server/models/job_validation/__tests__/job_validation.js:316:24)
22:21:09       at <anonymous>

@cjcenizal
Copy link
Contributor Author

Retest

…allout to indicate completion of import process when the user has opted to not import anything.
@cjcenizal cjcenizal force-pushed the enable-import-objects-test branch from ba121e7 to 5dd2541 Compare July 25, 2018 23:57
Copy link
Contributor

@nreese nreese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm
code review

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cjcenizal
Copy link
Contributor Author

Retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cjcenizal
Copy link
Contributor Author

Retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cjcenizal
Copy link
Contributor Author

Retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cjcenizal
Copy link
Contributor Author

Retest

1 similar comment
@cjcenizal
Copy link
Contributor Author

Retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cjcenizal
Copy link
Contributor Author

Retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

Copy link
Contributor

@bmcconaghy bmcconaghy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM. Please do fix the typo.

await PageObjects.settings.clickKibanaSavedObjects();

// Put in data which already exists
// This data has already been loaded by the "visualize" esArchive. We'll load it again
// so that we can overriding the existing visualization.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"override" instead of "overriding" here

@elasticmachine
Copy link
Contributor

💔 Build Failed

@cjcenizal
Copy link
Contributor Author

Looks like this is a new one:

19:14:20    │ proc  [ftr]        └-> should replace existing filter pill(s) when new item is selected
19:14:20    │ proc  [ftr]          └-> "before each" hook: global before each
19:14:20    │ proc  [ftr]          │ debg  clearComboBox for comboBoxSelector:listControlSelect0
19:14:20    │ proc  [ftr]          │ debg  TestSubjects.find(listControlSelect0)
19:14:20    │ proc  [ftr]          │ debg  findByCssSelector [data-test-subj~="listControlSelect0"]
19:14:30    │ proc  [ftr]          │ debg  TestSubjects.exists(comboBoxOptionsList)
19:14:30    │ proc  [ftr]          │ debg  existsByDisplayedByCssSelector [data-test-subj~="comboBoxOptionsList"]
19:14:31    │ proc  [ftr]          │ debg  TestSubjects.find(listControlSelect0)
19:14:31    │ proc  [ftr]          │ debg  findByCssSelector [data-test-subj~="listControlSelect0"]
19:14:31    │ proc  [ftr]          │ debg  ... sleep(500) start
19:14:31    │ proc  [ftr]          │ debg  ... sleep(500) end
19:14:31    │ proc  [ftr]          │ debg  clickByCssSelector(.euiComboBoxOption)
19:14:31    │ proc  [ftr]          │ debg  findByCssSelector .euiComboBoxOption
19:14:31    │ proc  [ftr]          │ debg  TestSubjects.exists(comboBoxOptionsList)
19:14:31    │ proc  [ftr]          │ debg  existsByDisplayedByCssSelector [data-test-subj~="comboBoxOptionsList"]
19:14:32    │ proc  [ftr]          │ debg  ... sleep(1000) start
19:14:33    │ proc  [ftr]          │ debg  ... sleep(1000) end
19:14:33    │ proc  [ftr]          │ debg  TestSubjects.click(inputControlSubmitBtn)
19:14:33    │ proc  [ftr]          │ debg  TestSubjects.find(inputControlSubmitBtn)
19:14:33    │ proc  [ftr]          │ debg  findByCssSelector [data-test-subj~="inputControlSubmitBtn"]
19:14:33    │ proc  [ftr]          │ debg  isGlobalLoadingIndicatorVisible
19:14:33    │ proc  [ftr]          │ debg  TestSubjects.exists(globalLoadingIndicator)
19:14:33    │ proc  [ftr]          │ debg  existsByDisplayedByCssSelector [data-test-subj~="globalLoadingIndicator"]
19:14:34    │ proc  [ftr]          │ debg  awaitGlobalLoadingIndicatorHidden
19:14:34    │ proc  [ftr]          │ debg  TestSubjects.find(globalLoadingIndicator-hidden)
19:14:34    │ proc  [ftr]          │ debg  findByCssSelector [data-test-subj~="globalLoadingIndicator-hidden"]
19:14:34    │ proc  [ftr]          │ debg  ... sleep(1000) start
19:14:35    │ proc  [ftr]          │ debg  ... sleep(1000) end
19:14:35    │ proc  [ftr]          │ debg  TestSubjects.exists(filter & filter-key-machine.os.raw & filter-value-ios & filter-enabled)
19:14:35    │ proc  [ftr]          │ debg  existsByDisplayedByCssSelector [data-test-subj~="filter"][data-test-subj~="filter-key-machine.os.raw"][data-test-subj~="filter-value-ios"][data-test-subj~="filter-enabled"]
19:14:35    │ proc  [ftr]          │ debg  TestSubjects.exists(filter & filter-key-machine.os.raw & filter-value-osx & filter-enabled)
19:14:35    │ proc  [ftr]          │ debg  existsByDisplayedByCssSelector [data-test-subj~="filter"][data-test-subj~="filter-key-machine.os.raw"][data-test-subj~="filter-value-osx"][data-test-subj~="filter-enabled"]
19:14:36    │ proc  [ftr]          │ info  Taking screenshot "/var/lib/jenkins/workspace/elastic+kibana+pull-request+multijob-selenium/kibana/test/functional/screenshots/failure/visualize app input control visualization updateFiltersOnChange is false should replace existing filter pill_s_ when new item is selected.png"
19:14:36    │ proc  [ftr]          │ info  Current URL is: http://localhost:5620/app/kibana#/visualize/create?type=input_control_vis&_g=(refreshInterval:(pause:!f,value:0),time:(from:%272017-01-01T00:00:00.000Z%27,mode:absolute,to:%272017-01-02T00:00:00.000Z%27))&_a=(filters:!((%27$state%27:(store:appState),meta:(alias:!n,controlledBy:%271532632446377%27,disabled:!f,index:%27logstash-*%27,key:machine.os.raw,negate:!f,params:(query:ios,type:phrase),type:phrase,value:ios),query:(match:(machine.os.raw:(query:ios,type:phrase))))),linked:!f,query:(language:lucene,query:%27%27),uiState:(),vis:(aggs:!(),params:(controls:!((fieldName:machine.os.raw,id:%271532632446377%27,indexPattern:%27logstash-*%27,label:%27%27,options:(dynamicOptions:!t,multiselect:!t,order:desc,size:5,type:terms),parent:%27%27,type:list)),pinFilters:!f,updateFiltersOnChange:!f,useTimeFilter:!f),title:%27New%20Visualization%27,type:input_control_vis))
19:14:36    │ proc  [ftr]          │ info  Saving page source to: /var/lib/jenkins/workspace/elastic+kibana+pull-request+multijob-selenium/kibana/test/functional/failure_debug/html/visualize app input control visualization updateFiltersOnChange is false should replace existing filter pill_s_ when new item is selected.html
19:14:36    │ proc  [ftr]        └- ✖ fail: "visualize app input control visualization updateFiltersOnChange is false should replace existing filter pill(s) when new item is selected"
19:14:36    │ proc  [ftr]        │        Error: expected true to equal false
19:14:36    │ proc  [ftr]        │         at Assertion.assert (node_modules/expect.js/index.js:96:13)
19:14:36    │ proc  [ftr]        │         at Assertion.be.Assertion.equal (node_modules/expect.js/index.js:216:10)
19:14:36    │ proc  [ftr]        │         at Context.it (test/functional/apps/visualize/_input_control_vis.js:92:33)
19:14:36    │ proc  [ftr]        │         at <anonymous>
19:14:36    │ proc  [ftr]        │         at process._tickCallback (internal/process/next_tick.js:188:7)

@cjcenizal
Copy link
Contributor Author

Retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cjcenizal
Copy link
Contributor Author

Retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cjcenizal
Copy link
Contributor Author

Retest

@elasticmachine
Copy link
Contributor

💚 Build Succeeded

@cjcenizal cjcenizal merged commit 5fcd10e into elastic:master Jul 27, 2018
@cjcenizal cjcenizal deleted the enable-import-objects-test branch July 27, 2018 01:23
cjcenizal added a commit that referenced this pull request Jul 27, 2018
* Reenable import objects tests. Refine their assertions. Add primary callout to indicate completion of import process when the user has opted to not import anything.
cjcenizal added a commit to cjcenizal/kibana that referenced this pull request Jul 27, 2018
* Reenable import objects tests. Refine their assertions. Add primary callout to indicate completion of import process when the user has opted to not import anything.
cjcenizal added a commit to cjcenizal/kibana that referenced this pull request Aug 27, 2018
* Reenable import objects tests. Refine their assertions. Add primary callout to indicate completion of import process when the user has opted to not import anything.
cjcenizal added a commit that referenced this pull request Aug 28, 2018
…tterns (#20379, #20379, #22068, #22029) (#22432)

* Add detection of invalid JSON searchSource to saved_object and dashboard (#20379)

* Reenable import objects tests (#21250)

* Reenable import objects tests. Refine their assertions. Add primary callout to indicate completion of import process when the user has opted to not import anything.

* fixing importing saved objects when there's a missing index pattern (#22068)

* fixing issue with importing vis with missing saved search (#22029)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants