search

swp-fu-eid / eid-fu-swp

Docker-based REST API implemented with Django and restframework.
MIT License
2 stars 1 forks source link

Setup CI #7

Closed m273d15 closed 7 years ago

m273d15 commented 7 years ago

Setup CI for project

m273d15 commented 7 years ago

I moved this repo to the new organisation swp-fu-eid and activated the travis ci via my private account for this repo. Now it should be enabled for any user. @Armagetron please verify this. Todo: We have to create a yaml configuration. Which requirements are the foundation to create the configuration?

BenjaminKeller commented 7 years ago

I think we need npm along with yasmine, frisby and jquery for eid client testing. Is that your question?

Armagetron commented 7 years ago

Travis should work now. I will setup a CI pipeline tomorrow.

BenjaminKeller commented 7 years ago

You can add my js-tests in branch check_for_eidclient. They are located in static/test/.

Armagetron commented 7 years ago

First steps in 8dbae111ec6dc5f33a68fbec74cab889e08c557f

Armagetron commented 7 years ago

The pipeline is running so far. However the communication between the different containers causes some trouble.

Maybe someone with more nodejs experience can look into this.

Armagetron commented 7 years ago

Current error is: Failed: request to https://eid.local/ failed, reason: getaddrinfo ENOTFOUND eid.local eid.local:443

Somehow the URL is not resolved.

BenjaminKeller commented 7 years ago

Try to add the following lines to the docker compose file:

    extra_hosts:
    - "eid.com:192.168.99.100"

I use this as my docker-compose.test.yml file, for example:

version: '2.1'

# mostly extending from main .yml
services:
  test-e2e:
    build: test/e2e
    restart: "no"
    environment:
    - BOILERPLATE_DOMAIN
    #- BOILERPLATE_IPV4_16PREFIX
    #- BOILERPLATE_IPV6_ADDRESS
    depends_on:
    - www
    networks:
    - front
    extra_hosts:
    - "eid.com:192.168.99.100"
Armagetron commented 7 years ago

I prepared a new branch (c1ea81d41285bcfdb2d71355a7327f6c9e7d2a7d) with a number of commits. It fails right now but once the other PR are merged it should build.

Armagetron commented 7 years ago

A PR (#29) has been created.

Armagetron commented 7 years ago

Merged