Skip to content
This repository has been archived by the owner on Nov 3, 2022. It is now read-only.

Update to clear page cache before each test run #204

Merged
merged 1 commit into from
Feb 21, 2018

Conversation

RyanMacG
Copy link
Contributor

@RyanMacG RyanMacG commented Feb 21, 2018

This protects against potential cache pollution from other tests that were
previously observed with Whitehall. As such we have switched these to
use the old method rather than the workaround

@RyanMacG RyanMacG requested a review from kevindew February 21, 2018 15:17
Copy link
Member

@kevindew kevindew 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 - you've got a couple of typos in the commit message.

@RyanMacG RyanMacG force-pushed the clear-page-cache-before-tests branch from 286ffd5 to 3d1ec25 Compare February 21, 2018 15:27
This protects against potential cache pollution from other tests that were
previously observed with Whitehall. As such we have switched these to
use the old method rather than the workaround
@RyanMacG RyanMacG force-pushed the clear-page-cache-before-tests branch from 3d1ec25 to f30c077 Compare February 21, 2018 15:28
@RyanMacG RyanMacG merged commit 7303236 into master Feb 21, 2018
@RyanMacG RyanMacG deleted the clear-page-cache-before-tests branch February 21, 2018 16:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants