opensource/galaxy
3
0
Fork 0
mirror of https://github.com/galaxyproject/galaxy.git synced 2026-10-30 18:13:23 +08:00
Code Issues Packages Projects Releases Wiki Activity

remove secondary test

Browse Source
This commit is contained in:
Cntient
2022-05-23 09:28:18 -04:00
parent 9c66a24985
commit f72a9086ec
1 changed files with 3 additions and 14 deletions
Download Patch File Download Diff File Expand all files Collapse all files
client/src/components/Workflow/Editor/Attributes.test.js
+3 -14
View File
@@ -7,7 +7,7 @@ jest.mock("app");
const TEST_ANNOTATION = "my cool annotation";
const TEST_NAME = "workflow_name";
const TEST_VERSIONS = [ //TODO (high) does it make sense to pre-fill this, if this is coming from prior user actions?
const TEST_VERSIONS = [
{ versions: 0, update_time: "2022-01-01", steps: 10 },
{ versions: 1, update_time: "2022-01-02", steps: 20 },
];
@@ -34,27 +34,16 @@ describe("Attributes", () => {
});
expect(wrapper.find(`[itemprop='description']`).attributes("content")).toBe(TEST_ANNOTATION);
expect(wrapper.find(`[itemprop='name']`).attributes("content")).toBe(TEST_NAME);
expect(wrapper.find(`#workflow-version-area > select`).exists()).toBeTruthy(); //TODO (low) does an `itemprop` need to be created in Attributes.vue (and why)?
expect(wrapper.find(`#workflow-version-area > select`).exists()).toBeTruthy();
const name = wrapper.find("#workflow-name");
expect(name.element.value).toBe(TEST_NAME);
await wrapper.setProps({ name: "new_workflow_name" });
expect(name.element.value).toBe("new_workflow_name");
const version = wrapper.findAll("#workflow-version-area > select > option");
console.log('version', version);
for (const v of version) {
console.log('v', v);
console.log('v.label', v.label);
const versionDate = (v.label).substring(versionLabel.indexOf(":") + 1, (v.label).indexOf(",")).trim();
expect(isDate(new Date(versionDate))).toBe(true);
}
const parameters = wrapper.findAll(".list-group-item"); //TODO (nit) not part of my fix, just curious: is this supposed to this exist in the code?
const parameters = wrapper.findAll(".list-group-item");
expect(parameters.length).toBe(2);
expect(parameters.at(0).text()).toBe("1: workflow_parameter_0");
expect(parameters.at(1).text()).toBe("2: workflow_parameter_1");
console.log(wrapper.html()); //TODO (low) we don't need this, right?
expect(wrapper.find("#workflow-annotation").element.value).toBe(TEST_ANNOTATION);
});
});