Skip to content

Commit

Permalink
EDSC-4329: Fixes tests
Browse files Browse the repository at this point in the history
  • Loading branch information
macrouch committed Jan 31, 2025
1 parent be02586 commit 6610be3
Show file tree
Hide file tree
Showing 5 changed files with 8 additions and 14 deletions.
4 changes: 1 addition & 3 deletions static/src/js/components/SearchPanels/SearchPanels.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -31,8 +31,6 @@ import { collectionSortKeys } from '../../constants/collectionSortKeys'
import AuthRequiredContainer from '../../containers/AuthRequiredContainer/AuthRequiredContainer'
import CollectionResultsBodyContainer
from '../../containers/CollectionResultsBodyContainer/CollectionResultsBodyContainer'
// Import CollectionDetailsBodyContainer
// from '../../containers/CollectionDetailsBodyContainer/CollectionDetailsBodyContainer'
import GranuleDetailsBodyContainer
from '../../containers/GranuleDetailsBodyContainer/GranuleDetailsBodyContainer'
import GranuleResultsBodyContainer
Expand Down Expand Up @@ -588,7 +586,7 @@ class SearchPanels extends PureComponent {
>
<PanelItem scrollable={false}>
<Suspense fallback={<Spinner type="dots" className="root__spinner spinner spinner--dots spinner--small" />}>
<CollectionDetailsBodyContainer />
<CollectionDetailsBodyContainer isActive />
</Suspense>
</PanelItem>
</PanelGroup>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -41,13 +41,9 @@ export const CollectionDetailsBodyContainer = ({
/>
)

CollectionDetailsBodyContainer.defaultProps = {
isActive: false
}

CollectionDetailsBodyContainer.propTypes = {
collectionMetadata: PropTypes.shape({}).isRequired,
isActive: PropTypes.bool,
isActive: PropTypes.bool.isRequired,
location: PropTypes.shape({}).isRequired,
onFocusedCollectionChange: PropTypes.func.isRequired,
onMetricsRelatedCollection: PropTypes.func.isRequired,
Expand Down
4 changes: 2 additions & 2 deletions tests/e2e/map/map_granules.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -386,7 +386,7 @@ test.describe('Map: Granule interactions', () => {
test('displays color map data to the user', async ({ page }) => {
await page.getByTestId('legend').hover({
position: {
x: 110,
x: 113,
y: 5
}
})
Expand Down Expand Up @@ -415,7 +415,7 @@ test.describe('Map: Granule interactions', () => {
await expect(page).toHaveScreenshot('colormap-2-screenshot.png', {
clip: {
x: 1138,
y: 263,
y: 260,
width: 252,
height: 47
}
Expand Down
2 changes: 1 addition & 1 deletion tests/e2e/panels/panels_behavior.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -87,6 +87,6 @@ test.describe('Panel Behavior', () => {
test('drags the panel to maximum width', async ({ page }) => {
await dragPanelToX(page, 1500)

await expect(page.getByTestId('panels-section')).toHaveCSS('width', '925px')
await expect(page.getByTestId('panels-section')).toHaveCSS('width', '916px')
})
})
6 changes: 3 additions & 3 deletions tests/e2e/tour/tour.spec.js
Original file line number Diff line number Diff line change
Expand Up @@ -347,9 +347,9 @@ test.describe('When not logged in', () => {

expectWithinMargin(spotlightRect, {
left: -10,
top: 185,
width: 329,
height: 697
top: 189,
width: 338,
height: 742
}, 10, 6)

// Step 7: Refine Search by Category
Expand Down

0 comments on commit 6610be3

Please sign in to comment.