Remove query obj from providers fab

[Prab-27]

Related discussion : here

This PR is part of the migration to SQLAlchemy 2.0

Removed session.query() from src and tests

---

^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

[potiuk]

conflicts to solve :(

[Prab-27]

I need to deep dive into pre-commit hook because it doesn't detect certain patterns session.query code from fab like

security_manager.sync_roles()
    num_pv_before = db.session().query(ab_perm_role).count()
    security_manager.sync_roles()
    num_pv_after = db.session().query(ab_perm_role).count()
    assert num_pv_before == num_pv_after

        sesh = self.appbuilder.get_session
        perms = sesh.query(Permission).filter(
            or_(
                Permission.action == None,  # noqa: E711
                Permission.resource == None,  # noqa: E711
            )
        )

[Prab-27]

Thanks I'll update pre-commit hook

[Prab-27]

Sorry! I forgot to include this earlier. It's updated now.

[vincbeck]

Nice! It detected some missing then (a good proof it works :))

[Prab-27]

Apologies—some query objects were still present. I should have reviewed them earlier. I'll update the code shortly