Skip to content

Commit 477ccfe

Browse files
Remove hard code tracks and subtracks
1 parent e7ee6d0 commit 477ccfe

File tree

3 files changed

+33
-9
lines changed

3 files changed

+33
-9
lines changed

__tests__/__snapshots__/index.js.snap

+10
Original file line numberDiff line numberDiff line change
@@ -377,6 +377,16 @@ Object {
377377
"DEVELOP": "Development",
378378
"QA": "Quality Assurance",
379379
},
380+
"OLD_COMPETITION_TRACKS": Object {
381+
"DATA_SCIENCE": "DATA_SCIENCE",
382+
"DESIGN": "DESIGN",
383+
"DEVELOP": "DEVELOP",
384+
"QA": "QA",
385+
},
386+
"OLD_SUBTRACKS": Object {
387+
"BUG_HUNT": "BUG_HUNT",
388+
"TEST_SUITES": "TEST_SUITES",
389+
},
380390
"REVIEW_OPPORTUNITY_TYPES": Object {
381391
"Contest Review": "Review",
382392
"Iterative Review": "Iterative Review",

src/services/reviewOpportunities.js

+11-9
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
* submitting applications.
55
*/
66
import _ from 'lodash';
7+
import { COMPETITION_TRACKS, OLD_COMPETITION_TRACKS, OLD_SUBTRACKS } from 'utils/tc';
78
import { getApi } from './api';
89

910
/**
@@ -16,17 +17,18 @@ export function normalizeChallenges(opportunities) {
1617
* until receive API V5 update. */
1718
_.map(opportunities, (opportunity) => {
1819
const { challenge } = opportunity;
19-
challenge.track = 'Development';
20+
challenge.track = COMPETITION_TRACKS.DEVELOP;
2021
if (challenge.technologies) {
21-
if (challenge.technologies.includes('Data Science')) {
22-
challenge.track = 'Data Science';
23-
} else if (challenge.technologies.includes('QA')) {
24-
challenge.track = 'Quality Assurance';
22+
if (challenge.technologies.includes(COMPETITION_TRACKS.DATA_SCIENCE)) {
23+
challenge.track = COMPETITION_TRACKS.DATA_SCIENCE;
24+
} else if (challenge.technologies.includes(OLD_COMPETITION_TRACKS.QA)) {
25+
challenge.track = COMPETITION_TRACKS.QA;
2526
}
26-
} else if (challenge.subTrack === 'TEST_SUITES' || challenge.subTrack === 'BUG_HUNT') {
27-
challenge.track = 'Quality Assurance';
28-
} else if (challenge.track === 'DESIGN') {
29-
challenge.track = 'Design';
27+
} else if (challenge.subTrack === OLD_SUBTRACKS.TEST_SUITES
28+
|| challenge.subTrack === OLD_SUBTRACKS.BUG_HUNT) {
29+
challenge.track = COMPETITION_TRACKS.QA;
30+
} else if (challenge.track === OLD_COMPETITION_TRACKS.DESIGN) {
31+
challenge.track = COMPETITION_TRACKS.DESIGN;
3032
}
3133
return _.defaults(opportunity, { challenge });
3234
});

src/utils/tc.js

+12
Original file line numberDiff line numberDiff line change
@@ -17,6 +17,18 @@ export const COMPETITION_TRACKS = {
1717
QA: 'Quality Assurance',
1818
};
1919

20+
export const OLD_COMPETITION_TRACKS = {
21+
DATA_SCIENCE: 'DATA_SCIENCE',
22+
DESIGN: 'DESIGN',
23+
DEVELOP: 'DEVELOP',
24+
QA: 'QA',
25+
};
26+
27+
export const OLD_SUBTRACKS = {
28+
TEST_SUITES: 'TEST_SUITES',
29+
BUG_HUNT: 'BUG_HUNT',
30+
};
31+
2032
/**
2133
* Review Opportunity types
2234
*/

0 commit comments

Comments
 (0)