Add flake8-pytest-style to linting

[gaugup]

Signed-off-by: Gaurav Gupta gaugup@microsoft.com

[gaugup]

@imatiach-msft, what does the test test_column_exception_without_brackets trying to test for DataMpper. Since all the code below is under pytest.raises so it fails for the wrong reason.

    def test_column_exception_without_brackets(self):
        with pytest.raises(ValueError):
            x = np.ones((2, 3))
            x[0, 0] = 0
            encoder = OneHotEncoder()
            encoder.fit(x[0])
            data_mapper = DataMapper([[0], encoder])
            data_mapper.transform(x)

correcting the code to below makes it fail.

    def test_column_exception_without_brackets(self):
        x = np.ones((2, 3))
        x[0, 0] = 0
        encoder = OneHotEncoder()
        encoder.fit(x)
        data_mapper = DataMapper([([0, 1, 2], encoder)])
        data_mapper.transform(x)
        with pytest.raises(ValueError):
            data_mapper.transform(x)

Not entirely sure what is this function trying to test? Shall we remove it?

[imatiach-msft]

@gaugup good idea, we can remove it

[gaugup]

test_column_exception_without_brackets

Thanks @imatiach-msft. Removed the test.