Skip to content

Commit

Permalink
chore: cleanup
Browse files Browse the repository at this point in the history
  • Loading branch information
tomivirkki committed Aug 27, 2021
1 parent 16f0196 commit 4f5a33c
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 17 deletions.
8 changes: 0 additions & 8 deletions packages/vaadin-combo-box/test/lazy-loading.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -576,7 +576,6 @@ describe('lazy loading', () => {

it('should set selectedItem matching value when items are loaded', () => {
comboBox.opened = true; // loads first page of dataProvider items
// comboBox.$.overlay.updateViewportBoundaries();
comboBox.$.overlay.ensureItemsRendered();
comboBox.value = 'item 0';
expect(comboBox.selectedItem).to.equal('item 0');
Expand Down Expand Up @@ -644,7 +643,6 @@ describe('lazy loading', () => {

it('should set matching selectedItem when items are loaded', () => {
comboBox.opened = true; // loads first page of dataProvider items
// comboBox.$.overlay.updateViewportBoundaries();
comboBox.$.overlay.ensureItemsRendered();
comboBox.value = 'value 0';
expect(comboBox.selectedItem).to.eql({ id: 0, value: 'value 0', label: 'label 0' });
Expand All @@ -667,7 +665,6 @@ describe('lazy loading', () => {
it('should select value matching selectedItem when items are loading', () => {
comboBox.selectedItem = 'item 0';
comboBox.opened = true;
// comboBox.$.overlay.updateViewportBoundaries();
comboBox.$.overlay.ensureItemsRendered();
expect(comboBox.value).to.equal('item 0');
const selectedRenderedItemElements = Array.from(
Expand All @@ -679,7 +676,6 @@ describe('lazy loading', () => {

it('should select value matching selectedItem when items are loaded', async () => {
comboBox.opened = true;
// comboBox.$.overlay.updateViewportBoundaries();
comboBox.$.overlay.ensureItemsRendered();
comboBox.selectedItem = 'item 0';
expect(comboBox.value).to.equal('item 0');
Expand Down Expand Up @@ -713,7 +709,6 @@ describe('lazy loading', () => {
it('should select value matching selectedItem when items are loading', () => {
comboBox.selectedItem = { id: 0, value: 'value 0', label: 'label 0' };
comboBox.opened = true;
// comboBox.$.overlay.updateViewportBoundaries();
comboBox.$.overlay.ensureItemsRendered();
expect(comboBox.value).to.equal('value 0');
const selectedRenderedItemElements = Array.from(
Expand All @@ -725,7 +720,6 @@ describe('lazy loading', () => {

it('should select value matching selectedItem when items are loaded', async () => {
comboBox.opened = true;
// comboBox.$.overlay.updateViewportBoundaries();
comboBox.$.overlay.ensureItemsRendered();
comboBox.selectedItem = { id: 0, value: 'value 0', label: 'label 0' };
expect(comboBox.value).to.equal('value 0');
Expand All @@ -750,7 +744,6 @@ describe('lazy loading', () => {
comboBox.selectedItem = { id: 0 };
comboBox.dataProvider = objectDataProvider;
comboBox.opened = true;
// comboBox.$.overlay.updateViewportBoundaries();
comboBox.$.overlay.ensureItemsRendered();
await nextFrame();
flush();
Expand All @@ -777,7 +770,6 @@ describe('lazy loading', () => {
);
};
comboBox.opened = true;
// comboBox.$.overlay.updateViewportBoundaries();
comboBox.$.overlay.ensureItemsRendered();
await nextFrame();
flush();
Expand Down
9 changes: 0 additions & 9 deletions packages/vaadin-combo-box/test/overlay-position.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -92,15 +92,6 @@ describe('overlay', () => {
expect(dropContentRect().top).to.be.closeTo(inputContentRect().bottom, 1);
});

it('should reposition on scroll', () => {
comboBox.opened = true;
// comboBox.$.overlay.updateViewportBoundaries = sinon.spy();

fire(document, 'scroll');

// expect(comboBox.$.overlay.updateViewportBoundaries.callCount).to.eql(1);
});

it('should be aligned with input container', () => {
comboBox.open();

Expand Down

0 comments on commit 4f5a33c

Please sign in to comment.