fuel-client unit tests require access to Nailgun API

Bug #1409726 reported by Roman Prykhodchenko
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Fuel for OpenStack
Fix Committed
Medium
Sylwester Brzeczkowski

Bug Description

Since python-fuelclient is a separate project it should be possible to run unit tests without any external dependencies to ensure the basic quality of the code.

Therefore all invocations of Nailgun API in unit tests should be mocked according to the specification of the API.

description: updated
Dmitry Pyzhov (dpyzhov)
summary: - Unit tests require access to Nailgun API
+ fuel-client unit tests require access to Nailgun API
Dima Shulyak (dshulyak)
Changed in fuel:
status: New → Triaged
Nikolay Markov (nmarkov)
Changed in fuel:
importance: High → Medium
Changed in fuel:
assignee: Fuel Python Team (fuel-python) → Sylwester Brzeczkowski (sbrzeczkowski)
Changed in fuel:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to python-fuelclient (master)

Fix proposed to branch: master
Review: https://review.openstack.org/165369

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to python-fuelclient (master)

Reviewed: https://review.openstack.org/165369
Committed: https://git.openstack.org/cgit/stackforge/python-fuelclient/commit/?id=9b0b6ab2f09ff75445550b7210104e21df5d8cd6
Submitter: Jenkins
Branch: master

commit 9b0b6ab2f09ff75445550b7210104e21df5d8cd6
Author: Sylwester Brzeczkowski <email address hidden>
Date: Fri Mar 20 12:55:02 2015 +0100

    run_tests.sh can be executed without Nailgun

    Splitted client's tests into unit and integration.
    Little cleanup in run_tests.sh
    Added new options in run_tests.sh:
    -i|--integration-tests
    -I|--no-integration-tests
    -u|--unit-tests
    -U|--no-unit-tests
    Script executes run_cleanup() and prepare_env() (which
    are responsible for cloning and running nailgun) only
    when running integration tests.

    Closes-Bug: #1409726
    Change-Id: I0685b7534b4a0684ffc932c7d24654548735c9b9

Changed in fuel:
status: In Progress → Fix Committed
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.