From e0091497e4b895037feca8373770a7acd983fe5f Mon Sep 17 00:00:00 2001 From: Sofiya Tepikin Date: Sat, 14 Sep 2019 21:03:03 +0200 Subject: [PATCH] Adds test for data deletion --- components/cycle-day/symptoms/symptom-view.js | 3 ++- components/header/delete-icon.js | 1 + e2e/dataInput.spec.js | 14 ++++++++++++++ 3 files changed, 17 insertions(+), 1 deletion(-) diff --git a/components/cycle-day/symptoms/symptom-view.js b/components/cycle-day/symptoms/symptom-view.js index c2a5dbc..749a873 100644 --- a/components/cycle-day/symptoms/symptom-view.js +++ b/components/cycle-day/symptoms/symptom-view.js @@ -35,9 +35,10 @@ class SymptomView extends Component { } componentDidUpdate() { + this.values = this.props.values const shouldShowDelete = this.checkIfHasValuesToDelete() if (shouldShowDelete !== this.state.shouldShowDelete) { - this.setState({ shouldShowDelete: this.checkIfHasValuesToDelete() }) + this.setState({ shouldShowDelete }) } } diff --git a/components/header/delete-icon.js b/components/header/delete-icon.js index ebcf9ba..1b67001 100644 --- a/components/header/delete-icon.js +++ b/components/header/delete-icon.js @@ -12,6 +12,7 @@ export default function DeleteIcon({ handleDelete }) { { console.log( 'This test a bit flaky. console.log apparently helps to fix it.' ) + + await goBack() + await expect(element(by.text(expectedSymptomSummary))).toExist() + + // Testing here additionally the deletion of data + await navigateToSymptomView(symptom) + await element(by.id('deleteIcon')).tap() + await element(by.text('YES, I AM SURE')).tap() + await expect(element(by.text(expectedSymptomSummary))).toNotExist() + + await navigateToSymptomView(symptom) + await enterTemperature() + break case 'note': await enterNote() @@ -127,4 +140,5 @@ describe('Symptom Data Input', () => { await expect(element(by.text(expectedSymptomSummary))).toExist() }) } + }) \ No newline at end of file