Skip to content

Fix remaining should syntax in timestamp specs#148

Merged
hownowstephen merged 1 commit into
mainfrom
fix-spec-issue
May 22, 2026
Merged

Fix remaining should syntax in timestamp specs#148
hownowstephen merged 1 commit into
mainfrom
fix-spec-issue

Conversation

@hownowstephen
Copy link
Copy Markdown
Contributor

@hownowstephen hownowstephen commented May 22, 2026

Summary

Test plan

  • CI passes all specs

Note

Low Risk
Low risk because changes are confined to RSpec expectation syntax in specs and do not affect runtime code paths.

Overview
Updates timestamp validation specs in spec/client_spec.rb to use modern RSpec expect { ... }.to raise_error syntax instead of deprecated lambda { ... }.should raise_error, fixing compatibility with current RSpec versions.

Reviewed by Cursor Bugbot for commit e00f582. Bugbot is set up for automated code reviews on this repo. Configure here.

Four specs still used `lambda { }.should raise_error` which fails
on modern RSpec. Convert to `expect { }.to raise_error`.
@hownowstephen hownowstephen merged commit 83e3e59 into main May 22, 2026
9 checks passed
@hownowstephen hownowstephen deleted the fix-spec-issue branch May 22, 2026 19:35
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