Skip to content

Commit 36b3262

Browse files
committed
chore: fix marker-utils tests
1 parent cb8d466 commit 36b3262

File tree

1 file changed

+16
-5
lines changed

1 file changed

+16
-5
lines changed

src/marker-utils.test.ts

Lines changed: 16 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,10 @@ const markerClasses = [
2323
google.maps.marker.AdvancedMarkerElement,
2424
];
2525

26+
beforeEach(() => {
27+
initialize();
28+
});
29+
2630
describe.each(markerClasses)(
2731
"MarkerUtils works with legacy and Advanced Markers",
2832
(markerClass) => {
@@ -35,10 +39,10 @@ describe.each(markerClasses)(
3539
test("identifies AdvancedMarker instances", () => {
3640
const isAdvancedMarker = MarkerUtils.isAdvancedMarker(new markerClass());
3741
if (markerClass === google.maps.marker.AdvancedMarkerElement) {
38-
expect(isAdvancedMarker).toBeTruthy;
42+
expect(isAdvancedMarker).toBeTruthy();
3943
return;
4044
}
41-
expect(isAdvancedMarker).toBeFalsy;
45+
expect(isAdvancedMarker).toBeFalsy();
4246
});
4347

4448
test("sets the map", () => {
@@ -50,7 +54,7 @@ describe.each(markerClasses)(
5054
).toEqual(map);
5155
return;
5256
}
53-
expect((marker as google.maps.Marker).setMap).toHaveBeenCalled;
57+
expect((marker as google.maps.Marker).setMap).toHaveBeenCalled();
5458
});
5559

5660
test("gets the marker position and returns a LatLng", () => {
@@ -67,9 +71,16 @@ describe.each(markerClasses)(
6771
});
6872
});
6973

70-
test("", () => {
74+
test(`${markerClass.name}.getVisible`, () => {
7175
const marker = new markerClass();
72-
expect(MarkerUtils.getVisible(marker)).toBeTruthy;
76+
77+
const res = MarkerUtils.getVisible(marker);
78+
79+
if (marker instanceof google.maps.Marker) {
80+
expect(marker.getVisible).toHaveBeenCalled();
81+
} else {
82+
expect(res).toBe(true);
83+
}
7384
});
7485
}
7586
);

0 commit comments

Comments
 (0)