Coder Social home page Coder Social logo

compliance-tests-ga4gh-tes's People

Contributors

yash-puligundla avatar

Stargazers

 avatar

Watchers

 avatar  avatar  avatar  avatar  avatar  avatar  avatar

compliance-tests-ga4gh-tes's Issues

Add tests for multiple executors

Problem

As per the current TES specification, multiple executors can be specified in a single request:

components:
  ...
  schemas:
    ...
    tesTask:
      ...
      properties:
        ...
        executors:
          type: array
          description: |-
            An array of executors to be run. Each of the executors will run one
            at a time sequentially. Each executor is a different command that
            will be run, and each can utilize a different docker image. But each of
            the executors will see the same mapped inputs and volumes that are declared
            in the parent CreateTask message.

            Execution stops on the first error.
          items:
            $ref: '#/components/schemas/tesExecutor'

Since TES v1.1, the behavior for the case in which one of multiple executors fails can further be modified with the ignore_error property of the tesExecutor schema:

components:
  ...
  schemas:
    ...
    tesExecutor:
      ...
      properties:
        ...
        ignore_error:
          type: boolean
          description: |-
            Default behavior of running an array of executors is that execution
            stops on the first error. If `ignore_error` is `True`, then the
            runner will record error exit codes, but will continue on to the next
            tesExecutor.

However, tests for multiple executors that checks whether a given TES instance processes multiple executors in the expected manner and whether the ignore_error flag is implemented correctly are currently not implemented.

Solution

  1. Add test to check whether a given TES instance is able to process multiple executors as expected (see above) (TES v1.0 and v1.1).
  2. Add test to check the correct implementation of the ignore_error property (TES v1.1 only).

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    ๐Ÿ–– Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. ๐Ÿ“Š๐Ÿ“ˆ๐ŸŽ‰

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google โค๏ธ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.