Skip to content

Disable perform_caching in DummyApp#4119

Merged
spaghetticode merged 2 commits intosolidusio:masterfrom
nebulab:4110-flaky-feature-spec-detected-disable-perform-caching-in-dummy-app
Jul 1, 2021
Merged

Disable perform_caching in DummyApp#4119
spaghetticode merged 2 commits intosolidusio:masterfrom
nebulab:4110-flaky-feature-spec-detected-disable-perform-caching-in-dummy-app

Conversation

@gsmendoza
Copy link
Copy Markdown
Contributor

@gsmendoza gsmendoza commented Jun 25, 2021

Description

In the test environment, we use the caching: true RSpec metadata to enable caching only on select specs. See core/lib/spree/testing_support/caching.rb.

Fixes #4110.

See #4110 (comment)

Checklist:

In the test environment, we use the `caching: true` RSpec metadata to
enable caching on select specs. See
core/lib/spree/testing_support/caching.rb.

Fixes solidusio#4110.
@gsmendoza gsmendoza marked this pull request as ready for review June 25, 2021 10:33
Comment thread core/lib/spree/testing_support/dummy_app.rb
Copy link
Copy Markdown
Member

@kennyadsl kennyadsl left a comment

Choose a reason for hiding this comment

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

Thanks George!

Copy link
Copy Markdown
Member

@spaghetticode spaghetticode left a comment

Choose a reason for hiding this comment

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

@gsmendoza thanks 👍

@spaghetticode spaghetticode merged commit 6a3d4b8 into solidusio:master Jul 1, 2021
@spaghetticode spaghetticode deleted the 4110-flaky-feature-spec-detected-disable-perform-caching-in-dummy-app branch July 1, 2021 05:43
waiting-for-dev added a commit to nebulab/solidus that referenced this pull request Sep 10, 2021
The new entries have been generated through the following command, as
detailed in the [how-to-release
guides](https://github.com/solidusio/solidus/wiki/How-to-release-Solidus#make-sure-changelogmd-is-up-to-date):

```bash
git rev-list v3.0.1...master | github_fast_changelog solidusio/solidus
```

The following items have been removed because of being change/revert
pairs or being changes on top of still-unreleased stuff:

- Docker: Change public keys server after deprecation
  [solidusio#4140](solidusio#4140)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Revert "Deprecate callback setting variant price from master"
  [solidusio#4132](solidusio#4132)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Deprecate callback setting variant price from master
  [solidusio#4078](solidusio#4078)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Remove staled comment
  [solidusio#4100](solidusio#4100)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Disable perform_caching in DummyApp
  [solidusio#4119](solidusio#4119)
  ([gsmendoza](https://github.com/gsmendoza))
- Master now tracks 3.1.0
  [solidusio#4033](solidusio#4033)
  ([kennyadsl](https://github.com/kennyadsl))
kennyadsl pushed a commit that referenced this pull request Sep 10, 2021
* Prepare CHANGELOG for Solidus 3.1 release

The new entries have been generated through the following command, as
detailed in the [how-to-release
guides](https://github.com/solidusio/solidus/wiki/How-to-release-Solidus#make-sure-changelogmd-is-up-to-date):

```bash
git rev-list v3.0.1...master | github_fast_changelog solidusio/solidus
```

The following items have been removed because of being change/revert
pairs or being changes on top of still-unreleased stuff:

- Docker: Change public keys server after deprecation
  [#4140](#4140)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Revert "Deprecate callback setting variant price from master"
  [#4132](#4132)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Deprecate callback setting variant price from master
  [#4078](#4078)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Remove staled comment
  [#4100](#4100)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Disable perform_caching in DummyApp
  [#4119](#4119)
  ([gsmendoza](https://github.com/gsmendoza))
- Master now tracks 3.1.0
  [#4033](#4033)
  ([kennyadsl](https://github.com/kennyadsl))

* Add backported entries
rmparr pushed a commit to rmparr/solidus that referenced this pull request Jun 1, 2022
rmparr pushed a commit to rmparr/solidus that referenced this pull request Jun 1, 2022
* Prepare CHANGELOG for Solidus 3.1 release

The new entries have been generated through the following command, as
detailed in the [how-to-release
guides](https://github.com/solidusio/solidus/wiki/How-to-release-Solidus#make-sure-changelogmd-is-up-to-date):

```bash
git rev-list v3.0.1...master | github_fast_changelog solidusio/solidus
```

The following items have been removed because of being change/revert
pairs or being changes on top of still-unreleased stuff:

- Docker: Change public keys server after deprecation
  [solidusio#4140](solidusio#4140)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Revert "Deprecate callback setting variant price from master"
  [solidusio#4132](solidusio#4132)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Deprecate callback setting variant price from master
  [solidusio#4078](solidusio#4078)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Remove staled comment
  [solidusio#4100](solidusio#4100)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Disable perform_caching in DummyApp
  [solidusio#4119](solidusio#4119)
  ([gsmendoza](https://github.com/gsmendoza))
- Master now tracks 3.1.0
  [solidusio#4033](solidusio#4033)
  ([kennyadsl](https://github.com/kennyadsl))

* Add backported entries
rmparr pushed a commit to rmparr/solidus that referenced this pull request Jun 1, 2022
rmparr pushed a commit to rmparr/solidus that referenced this pull request Jun 1, 2022
* Prepare CHANGELOG for Solidus 3.1 release

The new entries have been generated through the following command, as
detailed in the [how-to-release
guides](https://github.com/solidusio/solidus/wiki/How-to-release-Solidus#make-sure-changelogmd-is-up-to-date):

```bash
git rev-list v3.0.1...master | github_fast_changelog solidusio/solidus
```

The following items have been removed because of being change/revert
pairs or being changes on top of still-unreleased stuff:

- Docker: Change public keys server after deprecation
  [solidusio#4140](solidusio#4140)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Revert "Deprecate callback setting variant price from master"
  [solidusio#4132](solidusio#4132)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Deprecate callback setting variant price from master
  [solidusio#4078](solidusio#4078)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Remove staled comment
  [solidusio#4100](solidusio#4100)
  ([waiting-for-dev](https://github.com/waiting-for-dev))
- Disable perform_caching in DummyApp
  [solidusio#4119](solidusio#4119)
  ([gsmendoza](https://github.com/gsmendoza))
- Master now tracks 3.1.0
  [solidusio#4033](solidusio#4033)
  ([kennyadsl](https://github.com/kennyadsl))

* Add backported entries
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.

Flaky Feature Spec detected

4 participants