30

I have the following service: 

import { Injectable } from '@angular/core';

import { MenuItem } from './../classes/menu-item';
import { ITEMS } from './../static-data/items-list';

@Injectable()
export class ItemsListService {

    getItems(): Promise<MenuItem[]> {
        return Promise.resolve(ITEMS);
    }

}

The test for this service is here: 

import { TestBed, async, inject } from '@angular/core/testing';

import { ItemListService } from './item-list.service';
import { MenuItem } from './../classes/menu-item';
import { ITEMS } from './../static-data/items-list';

describe('ItemListService', () => {
  beforeEach(() => {
    TestBed.configureTestingModule({
        providers: [ ItemListService, MenuItem, ITEMS ]
    });
  });

  it('should ...', inject([ItemListService], (service: ItemListService) => {
    expect(service).toBeTruthy();
  }));
});

The MenuItem is defined here:

export class MenuItem {
    name: string;
    link: string;
}

ITEMS is defined here: import { MenuItem } from './../classes/menu-item';

export var ITEMS: MenuItem[] = [
    {name: 'Vehicles', link: '/vehicles'},
    {name: 'Gateways', link: '/gateways'},
    {name: 'Statuses', link: '/statuses'},
    {name: 'Logs', link: '/logs'}
]

When I run the test I am getting in the browsers console the followings errors:

FAILED ItemListService should ...

and

enter image description here

So why do I have these errors? And what is the solution for the test to work?

Mistalis
  • 17,793
  • 13
  • 73
  • 97
Cristian
  • 1,590
  • 5
  • 23
  • 38

5 Answers5

96

This is such an annoying error, thought I'd include another subtle cause to look for in your spec. In my case I specified providers instead of the correct value of provide as below

 TestBed.configureTestingModule({
      providers: [{provider: ApplicationActions, useClass: ActionMock}]

rather than offer useful information like "no 'provide' key specified" it simply reports

Failed: Invalid provider for the NgModule 'DynamicTestModule' - only instances of Provider and Type are allowed, got: [?[object Object]?, ...]

Craig
  • 1,383
  • 12
  • 9
  • 1
    and Check spelling and capitalization explicitly. When I was looking into this issue, it was because I used a capital letter instead of lowercase. `Provide` as opposed to `provide` – Fallenreaper Jan 13 '20 at 15:30
  • Nice catch, just for being explicit I add the correct syntax: `providers: [{provide: ApplicationActions, useClass: ActionMock}]` or `providers: [{provide: ApplicationActions, useValue: actionMockInstance}]` – nilsandrey Nov 30 '20 at 23:14
  • 1
    It's `provide` not `provider` – dataphile Apr 01 '21 at 06:25
  • 2
    I had this error because of an extra comma. – usalin Aug 29 '21 at 12:10
2

BTW also mind to use the class in providers, not some variable. This happened to me due to an accidental problematic replacement/casing:

Correct

 TestBed.configureTestingModule({
    // ...
    providers: [ SomeService ]
}

instead of...

Incorrect

 TestBed.configureTestingModule({
    // ...
    providers: [ someService ]
}

Note the camelCase variable (someService) is likely there if you use it in your test, that's why it does not throw a syntax error.

rklec
  • 73
  • 10
1

In my case, I had a stray comma in one of my provider lines, causing the DynamicTestModule to think I had passed an undefined definition.

        {
          provide: ApiService,
          useValue: {
            getUsers: jasmine
              .createSpy('getUsers')
              .and.returnValue(of({ status: 200, body: [] })),
          },
        },
        , // whoops!
        MessageService,
        { provide: Location, useValue: { back: jasmine.createSpy('back') } },
Kirk Sefchik
  • 673
  • 1
  • 9
  • 22
0

I got this error, because of an extra comma in the providers array.

Sivashankar
  • 493
  • 1
  • 8
  • 17
-2

I have same problem, when importing in my Ionic Framework project like this:

import {Device} from '@ionic-native/device'

instead of:

import {Device} from '@ionic-native/device/ngx'
ulou
  • 5,542
  • 5
  • 37
  • 47
DimaE
  • 1