diff options
| author | Kestred <kestred@riotcave.com> | 2014-08-23 22:27:52 -0700 |
|---|---|---|
| committer | Kestred <kestred@riotcave.com> | 2014-08-23 22:27:52 -0700 |
| commit | 18f954b31c242822b6c7430a50ca99a93adc6ea6 (patch) | |
| tree | b7ee4acac738d5b64b6fcfaf909756f784dd2d61 /spec/lib | |
| parent | 2eb3d91e1a6e51d3ca0be2fe0a5df750708b366d (diff) | |
| download | gitlab-ci-18f954b31c242822b6c7430a50ca99a93adc6ea6.tar.gz | |
Separate Commit model and logic from Build model|etc...
This is an entirely non-user facing change which prepares GitLab CI for future support of Parallel Builds.
See https://about.gitlab.com/2013/12/19/gitlab-ci-with-parallel-builds-and-deployments/.
These changes specifically avoid changing the supported API or changing any of the website views.
Changes to the website views will come in tandem with future features like "Multiple build scripts".
The supported API won't change as part of any future changes on this vein, to maintain support for the unofficial GitLab CI runners.
This closes the following implementation step:
1. A commit has many builds
Signed-off-by: Kestred <kestred@riotcave.com>
Diffstat (limited to 'spec/lib')
| -rw-r--r-- | spec/lib/charts_spec.rb | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/spec/lib/charts_spec.rb b/spec/lib/charts_spec.rb index f45a578..236cfc2 100644 --- a/spec/lib/charts_spec.rb +++ b/spec/lib/charts_spec.rb @@ -5,7 +5,8 @@ describe "Charts" do context "build_times" do before do @project = FactoryGirl.create(:project) - FactoryGirl.create(:build, :project_id => @project.id) + @commit = FactoryGirl.create(:commit, project: @project) + FactoryGirl.create(:build, commit: @commit) end it 'should return build times in minutes' do |