Skip to content

Commit

Permalink
Update route path
Browse files Browse the repository at this point in the history
For all other routes, we use the account/org syntax. For this one, I missed it and forgot to add ‘account’ for the route path. This fixes it
  • Loading branch information
scottybollinger committed Nov 17, 2020
1 parent 27dd373 commit 1eee0c7
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -411,7 +411,7 @@ describe('sources routes', () => {
});
});

describe('PUT /api/workplace_search/sources/{id}/searchable', () => {
describe('PUT /api/workplace_search/account/sources/{id}/searchable', () => {
let mockRouter: MockRouter;

beforeEach(() => {
Expand All @@ -421,7 +421,7 @@ describe('sources routes', () => {
it('creates a request handler', () => {
mockRouter = new MockRouter({
method: 'put',
path: '/api/workplace_search/sources/{id}/searchable',
path: '/api/workplace_search/account/sources/{id}/searchable',
payload: 'body',
});

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -268,7 +268,7 @@ export function registerAccountSourceSearchableRoute({
}: RouteDependencies) {
router.put(
{
path: '/api/workplace_search/sources/{id}/searchable',
path: '/api/workplace_search/account/sources/{id}/searchable',
validate: {
body: schema.object({
searchable: schema.boolean(),
Expand Down

0 comments on commit 1eee0c7

Please sign in to comment.