test(material/chips): prevent unintentional test success

[jeripeierSBB]

In 'MatChipInput' test, MatInputModule was not imported and therefore the matInput directive was not interpreted.
If importing MatInputModule into the testing module, the matInput directive gets applied and the test 'MatChipInput basic behavior should be aria-required if the list is required' would fail because of side effects by setting the aria-required attribute. Obviously, matInput does not remove the aria-required attribute, while the chips implementation is doing so (chips is interpreting false as null, while matInput leaves 'false' as-is).

[jeripeierSBB]

The mentioned test case of chip-input.spec.ts

    it('should be aria-required if the list is required', () => {
      expect(inputNativeElement.hasAttribute('aria-required')).toBe(false);

      fixture.componentInstance.required = true;
      fixture.detectChanges();

      expect(inputNativeElement.getAttribute('aria-required')).toBe('true');
    });

[mmalerba]

LGTM

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}