coinbase / temporal-ruby

Ruby SDK for Temporal
Apache License 2.0
238 stars 89 forks source link

Fix and add spec for local activities #247

Closed jeffschoner closed 1 year ago

jeffschoner commented 1 year ago

Summary

234 broke local activities because the Activity::Context constructor callsite they rely on was not updated. Local activities were also not covered by any specs, which made it easy to miss this. This change passes the correct number of parameters to the constructor and adds a basic spec for local activity invocation.

Fixes #245

Testing

New spec: bundle exec rspec spec/unit/lib/temporal/workflow/context_spec.rb:100