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

Use cache behavior tests from ActiveSupport #4

Merged
merged 31 commits into from
Mar 30, 2023
Merged

Conversation

hopsoft
Copy link
Owner

@hopsoft hopsoft commented Mar 20, 2023

Inspired by the work on #3

@jamiemccarthy
Copy link

Yeah, include'ing and overriding CacheStoreBehavior seems important to have in the test suite 👍

@github-advanced-security
Copy link

You have successfully added a new CodeQL configuration .github/workflows/codeql-analysis.yml:analyze/language:ruby. As part of the setup process, we have scanned this repository and found no existing alerts. In the future, you will see all code scanning alerts on the repository Security tab.

@hopsoft hopsoft marked this pull request as ready for review March 21, 2023 14:07
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
@hopsoft hopsoft requested review from jamiemccarthy and julianrubisch and removed request for julianrubisch and jamiemccarthy March 23, 2023 04:36
Copy link

@jamiemccarthy jamiemccarthy left a comment

Choose a reason for hiding this comment

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

Initial comments... I have some thoughts on CompositeCacheStore itself too that I'll submit separately...

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
Rakefile Show resolved Hide resolved
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
hopsoft and others added 3 commits March 26, 2023 11:36
Co-authored-by: Jamie McCarthy <jamie@mccarthy.vg>
Co-authored-by: Jamie McCarthy <jamie@mccarthy.vg>
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
lib/composite_cache_store.rb Outdated Show resolved Hide resolved
@hopsoft hopsoft requested review from julianrubisch and jamiemccarthy and removed request for julianrubisch March 30, 2023 02:08
Copy link

@jamiemccarthy jamiemccarthy left a comment

Choose a reason for hiding this comment

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

Looks good, this is a big step forward 👍

@hopsoft hopsoft merged commit 966850e into main Mar 30, 2023
7 checks passed
@hopsoft hopsoft deleted the hopsoft/behaviors branch March 30, 2023 14:04
@bf4 bf4 mentioned this pull request May 25, 2023
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.

None yet

3 participants