Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixes issues with running specs #2

Merged
merged 1 commit into from
Sep 21, 2018

Conversation

mscottford
Copy link
Collaborator

Without these changes, I was getting the output from below.

rake spec
/usr/local/bin/ruby -I/usr/local/bundle/gems/rspec-core-3.7.1/lib:/usr/local/bundle/gems/rspec-support-3.7.1/lib /usr/local/bundle/gems/rspec-core-3.7.1/exe/rspec --pattern spec/\*\*\{,/\*/\*\*\}/\*_spec.rb

An error occurred while loading ./spec/harvesting/client_spec.rb.
Failure/Error:
  RSpec.describe Harvesting::Client do
    let(:access_token) do
      "9999999.pt.abcdef"
    end
    let(:account_id) { "848484" }

    describe "#initialize" do
      context "when parameters are valid" do
        it "builds a client with a token and account" do
          Harvesting::Client.new(access_token: "foo", account_id: "bar")

NameError:
  uninitialized constant Harvesting::Client
# ./spec/harvesting/client_spec.rb:1:in `<top (required)>'

An error occurred while loading ./spec/harvesting/models/time_entry_spec.rb.
Failure/Error:
  RSpec.describe Harvesting::Models::TimeEntry, :vcr do
    let(:attrs) { Hash.new }
    let(:time_entry) { Harvesting::Models::TimeEntry.new(attrs) }
    let(:date) { "2018-05-14" }
    let(:started_time) { "8:00am" }
    let(:ended_time) { "9:00am" }
    let(:project_id) { '17367712' }
    let(:task_id) { '9713395' }
    let(:user_id) { '2108614' }

NameError:
  uninitialized constant Harvesting::Models
# ./spec/harvesting/models/time_entry_spec.rb:1:in `<top (required)>'


Finished in 0.00024 seconds (files took 0.16619 seconds to load)
0 examples, 0 failures, 2 errors occurred outside of examples

Copy link
Member

@etagwerker etagwerker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mscottford Cool, thanks! 👍

@etagwerker etagwerker merged commit 0f78797 into fastruby:master Sep 21, 2018
@mscottford mscottford deleted the spec-fixes branch December 21, 2018 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants