anisayakmitra-in opened a new pull request, #75544:
URL: https://github.com/apache/airflow/pull/75544
<!--
Licensed to the Apache Software Foundation (ASF) under one
or more contributor license agreements. See the NOTICE file
distributed with this work for additional information
regarding copyright ownership. The ASF licenses this file
to you under the Apache License, Version 2.0 (the
"License"); you may not use this file except in compliance
with the License. You may obtain a copy of the License at
http://www.apache.org/licenses/LICENSE-2.0
Unless required by applicable law or agreed to in writing,
software distributed under the License is distributed on an
"AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
KIND, either express or implied. See the License for the
specific language governing permissions and limitations
under the License.
-->
<!--
Thank you for contributing! Please make sure that your code changes
are covered with tests. And please also refer to the CONTRIBUTING.rst
and TEAMWORK guidelines for more information.
-->
### Description
Related to #72265.
Adds missing unit tests for auth manager login datamodels:
- `airflow.providers.amazon.aws.auth_manager.datamodels.login.LoginResponse`
(valid construction, dict validation, and required field enforcement)
-
`airflow.providers.fab.auth_manager.api_fastapi.datamodels.login.LoginResponse`
(valid construction, dict validation, and required field enforcement)
- Removes both modules from `OVERLOOKED_TESTS` in
`airflow-core/tests/unit/always/test_project_structure.py`.
### Use case/motivation
Improves provider test completeness for FastAPI auth manager endpoints and
reduces `OVERLOOKED_TESTS`.
### Related Issues
Closes: #72265
### Checklist
- [x] Are you creating a PR in a separate branch?
- [x] Did you check that there is no similar PR already open?
- [x] Have you written new tests for your core changes, as applicable?
- [x] Have you successfully run tests with your changes locally?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
