Skip to content

Add RSpec specs for Interaction, OutputGenerator, RequestContext, and UserInfo#8

Closed
mackuba wants to merge 3 commits into
masterfrom
codex/add-rspec-specs-for-multiple-classes
Closed

Add RSpec specs for Interaction, OutputGenerator, RequestContext, and UserInfo#8
mackuba wants to merge 3 commits into
masterfrom
codex/add-rspec-specs-for-multiple-classes

Conversation

@mackuba

@mackuba mackuba commented Jan 23, 2026

Copy link
Copy Markdown
Owner

Motivation

  • Add unit coverage for core feed/request objects to catch regressions in interaction mapping, output generation, request context handling, and user token parsing.
  • Conform tests to the requested project style using top-level describe, it "should ..." names, should matchers, and expect { ... }.to raise_error for exceptions.

Description

  • Added spec/blue_factory/interaction_spec.rb to validate Interaction attribute mapping and unknown-event handling.
  • Added spec/blue_factory/output_generator_spec.rb to test valid feed output, cursor/reqId rendering, pin/repost reasons, and invalid-response error cases.
  • Added spec/blue_factory/request_context_spec.rb and spec/blue_factory/user_info_spec.rb to cover environment access, UserInfo token parsing, JWT payload decoding, and authorization error handling.
  • Updated spec/spec_helper.rb to enable :should syntax and adjusted spec/blue_factory_spec.rb to use a top-level describe and should matcher.

Testing

  • Attempted to run bundle exec rspec, which failed in this environment with bundler: command not found: rspec, so the test suite could not be executed here.
  • All new specs were added under spec/blue_factory/ and committed; they are ready to run in an environment with the project dependencies installed.

Codex Task

@mackuba mackuba closed this Jan 24, 2026
@mackuba mackuba deleted the codex/add-rspec-specs-for-multiple-classes branch January 24, 2026 21:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant