diff --git a/client/src/app/instance/documentation/components/dataset-by-family.component.spec.ts b/client/src/app/instance/documentation/components/dataset-by-family.component.spec.ts index 02bc58e35024a8cc8fa6a971f428c311b1937a5c..460095fcc274611c032a581481a3ebc415477f51 100644 --- a/client/src/app/instance/documentation/components/dataset-by-family.component.spec.ts +++ b/client/src/app/instance/documentation/components/dataset-by-family.component.spec.ts @@ -19,48 +19,27 @@ const DATASET_LIST: Dataset[] = [ id_dataset_family: 1, public: true, full_data_path: '/data/path1', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'myDataset2', @@ -73,48 +52,27 @@ const DATASET_LIST: Dataset[] = [ id_dataset_family: 1, public: true, full_data_path: '/data/path2', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; const DATASET_FAMILY_LIST: DatasetFamily[] = [ diff --git a/client/src/app/instance/documentation/components/dataset-card-doc.component.spec.ts b/client/src/app/instance/documentation/components/dataset-card-doc.component.spec.ts index e9117ad082d9677566da8905f8a8c2d15e929540..b0730d6db407c9602a7dbadaa001ba2b7a035616 100644 --- a/client/src/app/instance/documentation/components/dataset-card-doc.component.spec.ts +++ b/client/src/app/instance/documentation/components/dataset-card-doc.component.spec.ts @@ -26,48 +26,27 @@ const DATASET: Dataset = { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }; describe('[Instance][Documentation][Component] DatasetCardDocComponent', () => { @@ -96,11 +75,4 @@ describe('[Instance][Documentation][Component] DatasetCardDocComponent', () => { it('should create the component', () => { expect(component).toBeDefined(); }); - - // it('#selectDataset() should navigate to the right page', () => { - // const spy = jest.spyOn(router, 'navigate'); - // component.selectDataset('myDataset'); - // expect(spy).toHaveBeenCalledTimes(1); - // expect(spy).toHaveBeenCalledWith(['/instance/myInstance/documentation/myDataset']); - // }); }); diff --git a/client/src/app/instance/search/components/result/download.component.spec.ts b/client/src/app/instance/search/components/result/download.component.spec.ts index dace40ba4e39bd0d7205c26cb89181ae887b2d8b..ad2261ff189ab55420d461d3aecc663925d2cf5a 100644 --- a/client/src/app/instance/search/components/result/download.component.spec.ts +++ b/client/src/app/instance/search/components/result/download.component.spec.ts @@ -43,48 +43,27 @@ describe('[Instance][Search][Component][Result] DownloadComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: false, - summary_opened: false - }, - server_link: { - server_link_enabled: false, - server_link_opened: true - }, - samp: { - samp_enabled: false, - samp_opened: false - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: false, + summary_opened: false, + server_link_enabled: false, + server_link_opened: true, + samp_enabled: false, + samp_opened: false, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -97,48 +76,27 @@ describe('[Instance][Search][Component][Result] DownloadComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: false, - download_opened: false, - download_csv: false, - download_ascii: false, - download_vo: false, - download_archive: false - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: false, + download_opened: false, + download_csv: false, + download_ascii: false, + download_vo: false, + download_archive: false, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; @@ -160,48 +118,27 @@ describe('[Instance][Search][Component][Result] DownloadComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: false, - summary_opened: false - }, - server_link: { - server_link_enabled: false, - server_link_opened: false - }, - samp: { - samp_enabled: false, - samp_opened: false - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: false, + summary_opened: false, + server_link_enabled: false, + server_link_opened: false, + samp_enabled: false, + samp_opened: false, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -214,48 +151,27 @@ describe('[Instance][Search][Component][Result] DownloadComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: false, - download_opened: false, - download_csv: false, - download_ascii: false, - download_vo: false, - download_archive: false - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: false, + download_opened: false, + download_csv: false, + download_ascii: false, + download_vo: false, + download_archive: false, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; @@ -277,48 +193,27 @@ describe('[Instance][Search][Component][Result] DownloadComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: false, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: false, - summary_opened: false - }, - server_link: { - server_link_enabled: false, - server_link_opened: false - }, - samp: { - samp_enabled: false, - samp_opened: false - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: false, + download_vo: true, + download_archive: true, + summary_enabled: false, + summary_opened: false, + server_link_enabled: false, + server_link_opened: false, + samp_enabled: false, + samp_opened: false, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -331,48 +226,27 @@ describe('[Instance][Search][Component][Result] DownloadComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: false, - download_opened: false, - download_csv: false, - download_ascii: false, - download_vo: false, - download_archive: false - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: false, + download_opened: false, + download_csv: false, + download_ascii: false, + download_vo: false, + download_archive: false, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; diff --git a/client/src/app/instance/search/components/result/reminder.component.spec.ts b/client/src/app/instance/search/components/result/reminder.component.spec.ts index 9a639c1faa5e5924b3113c8efec8ff67b5aad343..400bfa8313ec93e901cb05c7b201ffb92bb57c3c 100644 --- a/client/src/app/instance/search/components/result/reminder.component.spec.ts +++ b/client/src/app/instance/search/components/result/reminder.component.spec.ts @@ -42,48 +42,27 @@ describe('[Instance][Search][Component][Result] ReminderComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: false, - summary_opened: false - }, - server_link: { - server_link_enabled: false, - server_link_opened: true - }, - samp: { - samp_enabled: false, - samp_opened: false - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: false, + summary_opened: false, + server_link_enabled: false, + server_link_opened: true, + samp_enabled: false, + samp_opened: false, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -96,48 +75,27 @@ describe('[Instance][Search][Component][Result] ReminderComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; @@ -159,48 +117,27 @@ describe('[Instance][Search][Component][Result] ReminderComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: false, - summary_opened: false - }, - server_link: { - server_link_enabled: false, - server_link_opened: false - }, - samp: { - samp_enabled: false, - samp_opened: false - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: false, + summary_opened: false, + server_link_enabled: false, + server_link_opened: false, + samp_enabled: false, + samp_opened: false, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -213,48 +150,27 @@ describe('[Instance][Search][Component][Result] ReminderComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; diff --git a/client/src/app/instance/search/components/result/samp.component.spec.ts b/client/src/app/instance/search/components/result/samp.component.spec.ts index dfb4e4f0a797d978a06b58a76999b0de2021ad18..fd6f128fe38eb7a1600e348950a5a45001da36ee 100644 --- a/client/src/app/instance/search/components/result/samp.component.spec.ts +++ b/client/src/app/instance/search/components/result/samp.component.spec.ts @@ -38,48 +38,27 @@ describe('[Instance][Search][Component][Result] SampComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: false, - server_link_opened: true - }, - samp: { - samp_enabled: false, - samp_opened: false - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: false, + server_link_opened: true, + samp_enabled: false, + samp_opened: false, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -92,48 +71,27 @@ describe('[Instance][Search][Component][Result] SampComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; @@ -155,48 +113,27 @@ describe('[Instance][Search][Component][Result] SampComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: false, - server_link_opened: false - }, - samp: { - samp_enabled: false, - samp_opened: false - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: false, + server_link_opened: false, + samp_enabled: false, + samp_opened: false, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -209,48 +146,27 @@ describe('[Instance][Search][Component][Result] SampComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; diff --git a/client/src/app/instance/search/components/result/url-display.component.spec.ts b/client/src/app/instance/search/components/result/url-display.component.spec.ts index 5d47e352ebabfb43740812b11936e9fc6a507d07..7f7d3d748c60151b5b7e4d30a45b1ef5b2bff824 100644 --- a/client/src/app/instance/search/components/result/url-display.component.spec.ts +++ b/client/src/app/instance/search/components/result/url-display.component.spec.ts @@ -46,48 +46,27 @@ describe('[Instance][Search][Component][Result] UrlDisplayComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: false, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: false, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -100,48 +79,27 @@ describe('[Instance][Search][Component][Result] UrlDisplayComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; @@ -163,48 +121,27 @@ describe('[Instance][Search][Component][Result] UrlDisplayComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: false, - server_link_opened: false - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: false, + server_link_opened: false, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -217,48 +154,27 @@ describe('[Instance][Search][Component][Result] UrlDisplayComponent', () => { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image1'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; diff --git a/client/src/app/instance/search/components/summary.component.spec.ts b/client/src/app/instance/search/components/summary.component.spec.ts index 9d5c4d91f527d008f7c6a817d5f3cb5c8e440178..9681a02595c53e094ec5e04bad6d9beb43460175 100644 --- a/client/src/app/instance/search/components/summary.component.spec.ts +++ b/client/src/app/instance/search/components/summary.component.spec.ts @@ -42,48 +42,27 @@ describe('[Instance][Search][Component] SummaryComponent', () => { id_dataset_family: 1, public: true, full_data_path: '', - config: { - images: [], - survey: { - survey_enabled: true, - survey_label: '' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: '', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -96,48 +75,27 @@ describe('[Instance][Search][Component] SummaryComponent', () => { id_dataset_family: 1, public: true, full_data_path: '', - config: { - images: [], - survey: { - survey_enabled: true, - survey_label: '' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: '', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ]; component.datasetSelected = 'myDataset'; diff --git a/client/src/app/instance/store/effects/search-multiple.effects.spec.ts b/client/src/app/instance/store/effects/search-multiple.effects.spec.ts index ca8a91b29eb86ac1c0ba008d8b7a949da4f9a585..229b5d5ec68069c8541bcf9176ca13cd14947668 100644 --- a/client/src/app/instance/store/effects/search-multiple.effects.spec.ts +++ b/client/src/app/instance/store/effects/search-multiple.effects.spec.ts @@ -244,48 +244,27 @@ describe('[Instance][Store] SearchMultipleEffects', () => { id_dataset_family: 1, public: true, full_data_path: '', - config: { - images: [], - survey: { - survey_enabled: true, - survey_label: '' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: '', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true } ] ); diff --git a/client/src/test-data.ts b/client/src/test-data.ts index 61fdbda41a46f479ccdd1a6d9637370da1b712fe..2971799e89a1281ce4f57d9d0d6f5dd4e1e3ff8c 100644 --- a/client/src/test-data.ts +++ b/client/src/test-data.ts @@ -196,48 +196,27 @@ export const DATASET_LIST: Dataset[] = [ id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }, { name: 'anotherDataset', @@ -250,49 +229,28 @@ export const DATASET_LIST: Dataset[] = [ id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: false, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } - }, + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: false, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true + } ]; export const DATASET: Dataset = { @@ -306,48 +264,27 @@ export const DATASET: Dataset = { id_dataset_family: 1, public: true, full_data_path: '/data/path', - config: { - images: ['image'], - survey: { - survey_enabled: true, - survey_label: 'More about this survey' - }, - cone_search: { - cone_search_enabled: true, - cone_search_opened: true, - cone_search_column_ra: 1, - cone_search_column_dec: 2, - cone_search_plot_enabled: true, - cone_search_sdss_enabled: true, - cone_search_sdss_display: 1, - cone_search_background: [{ id: 1, enabled: true, display: 1 }] - }, - download: { - download_enabled: true, - download_opened: true, - download_csv: true, - download_ascii: true, - download_vo: true, - download_archive: true - }, - summary: { - summary_enabled: true, - summary_opened: true - }, - server_link: { - server_link_enabled: true, - server_link_opened: true - }, - samp: { - samp_enabled: true, - samp_opened: true - }, - datatable: { - datatable_enabled: true, - datatable_opened: true, - datatable_selectable_rows: true - } - } + info_survey_enabled: true, + info_survey_label: 'More about this survey', + cone_search_enabled: true, + cone_search_opened: true, + cone_search_column_ra: 1, + cone_search_column_dec: 2, + download_enabled: true, + download_opened: true, + download_csv: true, + download_ascii: true, + download_vo: true, + download_archive: true, + summary_enabled: true, + summary_opened: true, + server_link_enabled: true, + server_link_opened: true, + samp_enabled: true, + samp_opened: true, + datatable_enabled: true, + datatable_opened: true, + datatable_selectable_rows: true }; export const ATTRIBUTE_LIST: Attribute[] = [ diff --git a/server/src/Entity/Dataset.php b/server/src/Entity/Dataset.php index db0e655ef33efbd405b6aaa01108a94aebfd4c34..de128dbce30cea3c23fca0866015f6f43e905545 100644 --- a/server/src/Entity/Dataset.php +++ b/server/src/Entity/Dataset.php @@ -102,16 +102,16 @@ class Dataset implements \JsonSerializable protected $coneSearchOpened; /** - * @var string + * @var int * - * @Column(type="string", name="cone_search_column_ra", nullable=true) + * @Column(type="integer", name="cone_search_column_ra", nullable=true) */ protected $coneSearchColumnRa; /** - * @var string + * @var int * - * @Column(type="string", name="cone_search_column_dec", nullable=true) + * @Column(type="integer", name="cone_search_column_dec", nullable=true) */ protected $coneSearchColumnDec; diff --git a/server/src/Search/Query/ConeSearch.php b/server/src/Search/Query/ConeSearch.php index 4d4cee16257e82b9b412a6507e67c444f2aaa1a4..b0e514b098840ca64a31020858d99e71c00f2810 100644 --- a/server/src/Search/Query/ConeSearch.php +++ b/server/src/Search/Query/ConeSearch.php @@ -43,14 +43,13 @@ class ConeSearch extends AbstractQueryPart $ra = floatval($ra); $dec = floatval($dec); $radius = floatval($radius); - $coneSearchConfig = $dataset->getConfig()['cone_search']; - if ($coneSearchConfig['cone_search_enabled'] !== true) { + if ($dataset->getConeSearchEnabled() !== true) { throw SearchQueryException::coneSearchUnavailable(); } - $attributeRa = $this->getAttribute($dataset, $coneSearchConfig['cone_search_column_ra']); - $attributeDec = $this->getAttribute($dataset, $coneSearchConfig['cone_search_column_dec']); + $attributeRa = $this->getAttribute($dataset, $dataset->getConeSearchColumnRa()); + $attributeDec = $this->getAttribute($dataset, $dataset->getConeSearchColumnDec()); $columnRa = $dataset->getTableRef() . '.' . $attributeRa->getName(); $columnDec = $dataset->getTableRef() . '.' . $attributeDec->getName(); diff --git a/server/tests/Action/DatasetActionTest.php b/server/tests/Action/DatasetActionTest.php index f062fa390a8ba16468ecd8dbb663812781c271ec..e694344e74cb155cac93626b36b6d885d7b7555d 100644 --- a/server/tests/Action/DatasetActionTest.php +++ b/server/tests/Action/DatasetActionTest.php @@ -125,11 +125,29 @@ final class DatasetActionTest extends TestCase 'label' => 'Dataset1 label modified', 'description' => 'Dataset1 description', 'display' => 20, - 'count' => 5000, - 'vo' => false, 'data_path' => '/mnt/dataset1', - 'config' => '{}', 'public' => true, + 'info_survey_enabled' => true, + 'info_survey_label' => "More about this survey", + 'cone_search_enabled' => false, + 'cone_search_opened' => true, + 'cone_search_column_ra' => null, + 'cone_search_column_dec' => null, + 'download_enabled' => true, + 'download_opened' => false, + 'download_csv' => true, + 'download_ascii' => true, + 'download_vo' => false, + 'download_archive' => true, + 'summary_enabled' => true, + 'summary_opened' => false, + 'server_link_enabled' => false, + 'server_link_opened' => false, + 'samp_enabled' => false, + 'samp_opened' => false, + 'datatable_enabled' => true, + 'datatable_opened' => false, + 'datatable_selectable_rows' => false, 'id_dataset_family' => 1 ); } diff --git a/server/tests/Action/DatasetListActionTest.php b/server/tests/Action/DatasetListActionTest.php index 51a5a343ceb1d6b63fc53ac3346fd35117c183df..e9627003c6ba10def65ad3f9916865326f38366f 100644 --- a/server/tests/Action/DatasetListActionTest.php +++ b/server/tests/Action/DatasetListActionTest.php @@ -121,11 +121,29 @@ final class DatasetListActionTest extends TestCase 'label' => 'Dataset1 label', 'description' => 'Dataset1 description', 'display' => 10, - 'count' => 10000, - 'vo' => false, 'data_path' => '/mnt/dataset1', - 'config' => '{}', 'public' => true, + 'info_survey_enabled' => true, + 'info_survey_label' => "More about this survey", + 'cone_search_enabled' => false, + 'cone_search_opened' => true, + 'cone_search_column_ra' => null, + 'cone_search_column_dec' => null, + 'download_enabled' => true, + 'download_opened' => false, + 'download_csv' => true, + 'download_ascii' => true, + 'download_vo' => false, + 'download_archive' => true, + 'summary_enabled' => true, + 'summary_opened' => false, + 'server_link_enabled' => false, + 'server_link_opened' => false, + 'samp_enabled' => false, + 'samp_opened' => false, + 'datatable_enabled' => true, + 'datatable_opened' => false, + 'datatable_selectable_rows' => false, 'survey_name' => 'anis_survey', 'id_dataset_family' => 1 ); diff --git a/server/tests/Handlers/LogErrorHandlerTest.php b/server/tests/Handlers/LogErrorHandlerTest.php index c70c0f9c55c1498b28ce885e2f7528870c5247b7..66ed02ac06f5beea408354a9f4fe15990588a232 100644 --- a/server/tests/Handlers/LogErrorHandlerTest.php +++ b/server/tests/Handlers/LogErrorHandlerTest.php @@ -41,7 +41,7 @@ final class LogErrorHandlerTest extends TestCase { $logger = $this->createMock(LoggerInterface::class); $logger->expects($this->once()) - ->method('info'); + ->method('error'); $this->logErrorHandler->setLogger($logger); $logError = self::getMethod('logError'); diff --git a/server/tests/Search/Query/ConeSearchTest.php b/server/tests/Search/Query/ConeSearchTest.php index c0bf21091ffbe35f43c1b995e308cfce80175121..dda9d59f6f0410e530383a152821029e84ed7441 100644 --- a/server/tests/Search/Query/ConeSearchTest.php +++ b/server/tests/Search/Query/ConeSearchTest.php @@ -32,12 +32,10 @@ final class ConeSearchTest extends TestCase $dec = $this->createMock(Attribute::class); $dec->method('getId')->willReturn(3); $datasetSelected = $this->createMock(Dataset::class); + $datasetSelected->method('getConeSearchEnabled')->willReturn(true); + $datasetSelected->method('getConeSearchColumnRa')->willReturn(2); + $datasetSelected->method('getConeSearchColumnDec')->willReturn(3); $datasetSelected->method('getAttributes')->willReturn(array($id, $ra, $dec)); - $datasetSelected->method('getConfig')->willReturn(array('cone_search' => array( - 'cone_search_enabled' => true, - 'cone_search_column_ra' => 2, - 'cone_search_column_dec' => 3 - ))); $doctrineQueryBuilder = $this->createMock(DoctrineQueryBuilder::class); $expr = $this->createMock(ExpressionBuilder::class); @@ -63,11 +61,8 @@ final class ConeSearchTest extends TestCase $this->expectException(SearchQueryException::class); $anisQueryBuilder = $this->createMock(AnisQueryBuilder::class); $datasetSelected = $this->createMock(Dataset::class); - $datasetSelected->method('getConfig')->willReturn(array('cone_search' => array( - 'cone_search_enabled' => false, - 'cone_search_column_ra' => 2, - 'cone_search_column_dec' => 3 - ))); + $datasetSelected->method('getConeSearchEnabled')->willReturn(false); + $queryParams = array('cs' => '102.5:0.0:100'); (new ConeSearch())($anisQueryBuilder, $datasetSelected, $queryParams); } @@ -81,12 +76,10 @@ final class ConeSearchTest extends TestCase $dec = $this->createMock(Attribute::class); $dec->method('getId')->willReturn(3); $datasetSelected = $this->createMock(Dataset::class); + $datasetSelected->method('getConeSearchEnabled')->willReturn(true); + $datasetSelected->method('getConeSearchColumnRa')->willReturn(2); + $datasetSelected->method('getConeSearchColumnDec')->willReturn(3); $datasetSelected->method('getAttributes')->willReturn(array($id, $ra, $dec)); - $datasetSelected->method('getConfig')->willReturn(array('cone_search' => array( - 'cone_search_enabled' => true, - 'cone_search_column_ra' => 2, - 'cone_search_column_dec' => 3 - ))); $doctrineQueryBuilder = $this->createMock(DoctrineQueryBuilder::class); $expr = $this->createMock(ExpressionBuilder::class);