Fixed curations (#117297)

This commit is contained in:
Jason Stoltzfus 2021-11-03 11:02:14 -04:00 committed by GitHub
parent 6cf9f8c966
commit 9e02087bd7
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
10 changed files with 30 additions and 30 deletions

View file

@ -28,7 +28,7 @@ const MOCK_VALUES = {
},
// EngineLogic
engine: {
search_relevance_suggestions_active: true,
adaptive_relevance_suggestions_active: true,
},
};
@ -53,7 +53,7 @@ describe('SuggestedDocumentsCallout', () => {
});
it('is empty when suggestions are not active', () => {
const values = set('engine.search_relevance_suggestions_active', false, MOCK_VALUES);
const values = set('engine.adaptive_relevance_suggestions_active', false, MOCK_VALUES);
setMockValues(values);
const wrapper = shallow(<SuggestedDocumentsCallout />);

View file

@ -22,13 +22,13 @@ export const SuggestedDocumentsCallout: React.FC = () => {
curation: { suggestion, queries },
} = useValues(CurationLogic);
const {
engine: { search_relevance_suggestions_active: searchRelevanceSuggestionsActive },
engine: { adaptive_relevance_suggestions_active: adaptiveRelevanceSuggestionsActive },
} = useValues(EngineLogic);
if (
typeof suggestion === 'undefined' ||
suggestion.status !== 'pending' ||
searchRelevanceSuggestionsActive === false
adaptiveRelevanceSuggestionsActive === false
) {
return null;
}

View file

@ -55,7 +55,7 @@ describe('Curations', () => {
},
// EngineLogic
engine: {
search_relevance_suggestions_active: true,
adaptive_relevance_suggestions_active: true,
},
};
@ -89,7 +89,7 @@ describe('Curations', () => {
});
it('renders less tabs when suggestions are not active', () => {
setMockValues(set('engine.search_relevance_suggestions_active', false, values));
setMockValues(set('engine.adaptive_relevance_suggestions_active', false, values));
const wrapper = shallow(<Curations />);
expect(getPageTitle(wrapper)).toEqual('Curated results');
@ -99,7 +99,7 @@ describe('Curations', () => {
});
it('renders a New! badge when suggestions are not active', () => {
setMockValues(set('engine.search_relevance_suggestions_active', false, values));
setMockValues(set('engine.adaptive_relevance_suggestions_active', false, values));
const wrapper = shallow(<Curations />);
expect(getPageTitle(wrapper)).toEqual('Curated results');
@ -109,7 +109,7 @@ describe('Curations', () => {
});
it('hides the badge when suggestions are active', () => {
setMockValues(set('engine.search_relevance_suggestions_active', true, values));
setMockValues(set('engine.adaptive_relevance_suggestions_active', true, values));
const wrapper = shallow(<Curations />);
expect(getPageTitle(wrapper)).toEqual('Curated results');

View file

@ -30,10 +30,10 @@ export const Curations: React.FC = () => {
const { dataLoading, meta, selectedPageTab } = useValues(CurationsLogic);
const { loadCurations, onSelectPageTab } = useActions(CurationsLogic);
const {
engine: { search_relevance_suggestions_active: searchRelevanceSuggestionsActive },
engine: { adaptive_relevance_suggestions_active: adaptiveRelevanceSuggestionsActive },
} = useValues(EngineLogic);
const suggestionsEnabled = searchRelevanceSuggestionsActive;
const suggestionsEnabled = adaptiveRelevanceSuggestionsActive;
const OVERVIEW_TAB = {
label: i18n.translate(
@ -72,7 +72,7 @@ export const Curations: React.FC = () => {
),
};
const pageTabs = searchRelevanceSuggestionsActive
const pageTabs = adaptiveRelevanceSuggestionsActive
? [OVERVIEW_TAB, HISTORY_TAB, SETTINGS_TAB]
: [OVERVIEW_TAB, SETTINGS_TAB];

View file

@ -36,7 +36,7 @@ const MOCK_VALUES = {
],
// EngineLogic
engine: {
search_relevance_suggestions_active: true,
adaptive_relevance_suggestions_active: true,
},
};
@ -71,14 +71,14 @@ describe('CurationsOverview', () => {
});
it('renders a suggestions table when suggestions are active', () => {
setMockValues(set('engine.search_relevance_suggestions_active', true, MOCK_VALUES));
setMockValues(set('engine.adaptive_relevance_suggestions_active', true, MOCK_VALUES));
const wrapper = shallow(<CurationsOverview />);
expect(wrapper.find(SuggestionsTable).exists()).toBe(true);
});
it('doesn\t render a suggestions table when suggestions are not active', () => {
setMockValues(set('engine.search_relevance_suggestions_active', false, MOCK_VALUES));
setMockValues(set('engine.adaptive_relevance_suggestions_active', false, MOCK_VALUES));
const wrapper = shallow(<CurationsOverview />);
expect(wrapper.find(SuggestionsTable).exists()).toBe(false);

View file

@ -19,10 +19,10 @@ import { CurationsLogic } from '../curations_logic';
export const CurationsOverview: React.FC = () => {
const { curations } = useValues(CurationsLogic);
const {
engine: { search_relevance_suggestions_active: searchRelevanceSuggestionsActive },
engine: { adaptive_relevance_suggestions_active: adaptiveRelevanceSuggestionsActive },
} = useValues(EngineLogic);
const shouldShowSuggestions = searchRelevanceSuggestionsActive;
const shouldShowSuggestions = adaptiveRelevanceSuggestionsActive;
return (
<>

View file

@ -43,7 +43,7 @@ describe('EngineLogic', () => {
schema: { test: SchemaType.Text },
apiTokens: [],
apiKey: 'some-key',
search_relevance_suggestions_active: true,
adaptive_relevance_suggestions_active: true,
};
const DEFAULT_VALUES = {

View file

@ -53,8 +53,8 @@ export interface EngineDetails extends Engine {
isMeta: boolean;
engine_count?: number;
includedEngines?: EngineDetails[];
search_relevance_suggestions?: SearchRelevanceSuggestionDetails;
search_relevance_suggestions_active: boolean;
adaptive_relevance_suggestions?: SearchRelevanceSuggestionDetails;
adaptive_relevance_suggestions_active: boolean;
}
interface ResultField {

View file

@ -19,12 +19,12 @@ import { SuggestedCurationsCallout } from './suggested_curations_callout';
const MOCK_VALUES = {
engine: {
search_relevance_suggestions: {
adaptive_relevance_suggestions: {
curation: {
pending: 1,
},
},
search_relevance_suggestions_active: true,
adaptive_relevance_suggestions_active: true,
},
};
@ -44,7 +44,7 @@ describe('SuggestedCurationsCallout', () => {
setMockValues({
...MOCK_VALUES,
engine: {
search_relevance_suggestions_active: true,
adaptive_relevance_suggestions_active: true,
},
});
@ -54,7 +54,7 @@ describe('SuggestedCurationsCallout', () => {
});
it('is empty when suggestions are not active', () => {
const values = set('engine.search_relevance_suggestions_active', false, MOCK_VALUES);
const values = set('engine.adaptive_relevance_suggestions_active', false, MOCK_VALUES);
setMockValues(values);
const wrapper = shallow(<SuggestedCurationsCallout />);
@ -63,7 +63,7 @@ describe('SuggestedCurationsCallout', () => {
});
it('is empty when no pending curations', () => {
const values = set('engine.search_relevance_suggestions.curation.pending', 0, MOCK_VALUES);
const values = set('engine.adaptive_relevance_suggestions.curation.pending', 0, MOCK_VALUES);
setMockValues(values);
const wrapper = shallow(<SuggestedCurationsCallout />);

View file

@ -17,17 +17,17 @@ import { EngineLogic, generateEnginePath } from '../../engine';
export const SuggestedCurationsCallout: React.FC = () => {
const {
engine: {
search_relevance_suggestions: searchRelevanceSuggestions,
search_relevance_suggestions_active: searchRelevanceSuggestionsActive,
adaptive_relevance_suggestions: adaptiveRelevanceSuggestions,
adaptive_relevance_suggestions_active: adaptiveRelevanceSuggestionsActive,
},
} = useValues(EngineLogic);
const pendingCount = searchRelevanceSuggestions?.curation.pending;
const pendingCount = adaptiveRelevanceSuggestions?.curation.pending;
if (
typeof searchRelevanceSuggestions === 'undefined' ||
typeof adaptiveRelevanceSuggestions === 'undefined' ||
pendingCount === 0 ||
searchRelevanceSuggestionsActive === false
adaptiveRelevanceSuggestionsActive === false
) {
return null;
}
@ -46,7 +46,7 @@ export const SuggestedCurationsCallout: React.FC = () => {
}
)}
buttonTo={generateEnginePath(ENGINE_CURATIONS_PATH)}
lastUpdatedTimestamp={searchRelevanceSuggestions.curation.last_updated}
lastUpdatedTimestamp={adaptiveRelevanceSuggestions.curation.last_updated}
/>
);
};