Merge branch 'master' of github.com:Human-Connection/Human-Connection into 781-language-of-contribution

This commit is contained in:
Matt Rider 2019-06-13 17:30:24 -03:00
commit 8a214a4007
4 changed files with 18 additions and 4 deletions

View File

@ -21,6 +21,9 @@ install:
- wait-on http://localhost:7474 - wait-on http://localhost:7474
script: script:
- export CYPRESS_RETRIES=1
- export BRANCH=$(if [ "$TRAVIS_PULL_REQUEST" == "false" ]; then echo $TRAVIS_BRANCH; else echo $TRAVIS_PULL_REQUEST_BRANCH; fi)
- echo "TRAVIS_BRANCH=$TRAVIS_BRANCH, PR=$PR, BRANCH=$BRANCH"
# Backend # Backend
- docker-compose exec backend yarn run lint - docker-compose exec backend yarn run lint
- docker-compose exec backend yarn run test:jest --ci --verbose=false --coverage - docker-compose exec backend yarn run test:jest --ci --verbose=false --coverage
@ -34,7 +37,7 @@ script:
- docker-compose exec webapp yarn run test --ci --verbose=false --coverage - docker-compose exec webapp yarn run test --ci --verbose=false --coverage
- docker-compose exec -d backend yarn run test:before:seeder - docker-compose exec -d backend yarn run test:before:seeder
# Fullstack # Fullstack
- CYPRESS_RETRIES=1 yarn run cypress:run - yarn run cypress:run
# Coverage # Coverage
- codecov - codecov

View File

@ -24,4 +24,5 @@ RUN yarn run build
FROM base as production FROM base as production
ENV NODE_ENV=production ENV NODE_ENV=production
COPY --from=builder /nitro-backend/dist ./dist COPY --from=builder /nitro-backend/dist ./dist
COPY ./public/img/ ./public/img/
RUN yarn install --frozen-lockfile --non-interactive RUN yarn install --frozen-lockfile --non-interactive

View File

@ -6,8 +6,11 @@ const legacyUrls = [
export const fixUrl = url => { export const fixUrl = url => {
legacyUrls.forEach(legacyUrl => { legacyUrls.forEach(legacyUrl => {
url = url.replace(legacyUrl, '/api') url = url.replace(legacyUrl, '')
}) })
if (!url.startsWith('/')) {
url = `/${url}`
}
return url return url
} }

View File

@ -1,12 +1,19 @@
import { fixImageURLs } from './fixImageUrlsMiddleware' import { fixImageURLs } from './fixImageUrlsMiddleware'
describe('fixImageURLs', () => { describe('fixImageURLs', () => {
describe('edge case: image url is exact match of legacy url', () => {
it('replaces it with `/`', () => {
const url = 'https://api-alpha.human-connection.org'
expect(fixImageURLs(url)).toEqual('/')
})
})
describe('image url of legacy alpha', () => { describe('image url of legacy alpha', () => {
it('removes domain', () => { it('removes domain', () => {
const url = const url =
'https://api-alpha.human-connection.org/uploads/4bfaf9172c4ba03d7645108bbbd16f0a696a37d01eacd025fb131e5da61b15d9.png' 'https://api-alpha.human-connection.org/uploads/4bfaf9172c4ba03d7645108bbbd16f0a696a37d01eacd025fb131e5da61b15d9.png'
expect(fixImageURLs(url)).toEqual( expect(fixImageURLs(url)).toEqual(
'/api/uploads/4bfaf9172c4ba03d7645108bbbd16f0a696a37d01eacd025fb131e5da61b15d9.png', '/uploads/4bfaf9172c4ba03d7645108bbbd16f0a696a37d01eacd025fb131e5da61b15d9.png',
) )
}) })
}) })
@ -16,7 +23,7 @@ describe('fixImageURLs', () => {
const url = const url =
'https://staging-api.human-connection.org/uploads/1b3c39a24f27e2fb62b69074b2f71363b63b263f0c4574047d279967124c026e.jpeg' 'https://staging-api.human-connection.org/uploads/1b3c39a24f27e2fb62b69074b2f71363b63b263f0c4574047d279967124c026e.jpeg'
expect(fixImageURLs(url)).toEqual( expect(fixImageURLs(url)).toEqual(
'/api/uploads/1b3c39a24f27e2fb62b69074b2f71363b63b263f0c4574047d279967124c026e.jpeg', '/uploads/1b3c39a24f27e2fb62b69074b2f71363b63b263f0c4574047d279967124c026e.jpeg',
) )
}) })
}) })