PyCQA / astroid

90%

main: 92%

LAST BUILD ON BRANCH boto3-transform

branch: boto3-transform

CHANGE BRANCH

x

Reset

• boto3-transform
• 1.6
• 2.0
• 2.10
• 2.2
• 2.3
• 2.4
• 2.5
• 2.5.2
• 2.6.6-release-task
• 2.9
• add-autoflake-to-pre-commit
• add-dateutil-to-requirements
• add-flake8-in-pre-commit-configuration
• add-github-actions
• add-isort-pylint-autoflake-to-pre-commit
• add-isort-to-pre-commit
• add-property-support
• always-pick-same-parsing-module
• astroid-1.5.0
• astroid-1.5.1
• astroid-1.5.2
• astroid-1.5.3
• astroid-1.6.6
• astroid-2.1.0
• astroid-2.2.0
• astroid-2.2.1
• astroid-2.2.3
• astroid-2.2.4
• astroid-2.2.5
• astroid-2.3.0
• astroid-2.3.1
• astroid-2.3.2
• astroid-2.3.3
• astroid-2.4.0
• astroid-2.4.1
• astroid-2.4.2
• astroid-2.5
• astroid-2.5.1
• astroid-2.5.2
• astroid-2.5.3
• bug_666
• bug_666_pure
• bug_pylint_2885
• bug_pylint_3208
• bug_pylint_3319
• enable-useless-supression
• fix-coverage
• fix-crash-in-dunder-inference
• fix-crash-lookup
• fix-crash-uninferable
• fix-incorrect-parameter-passing-in-partial
• fix-is-generator-for-assign-nodes
• fix-is-generator-yield
• fix-property-generated-func
• fix-pylint-legacy-warnings
• fix-pylint-warning-from-master-branch
• fix-super-property-inference
• fix-wrapped-c-object-deleted
• fix_665
• fix_851
• fix_pylint_1628
• fix_pylint_73
• import-system-cwd-removed
• infer-qualified-builtins-classmethod
• main
• master
• move_tests
• positional-only-arguments
• prepare-for-2.5.3
• remove-enum34-from-dependencies
• remove-license-duplication
• revert-397-import_all
• simplification-for-python3.6-code
• skip-starred-in-enum
• stop-installing-tests-with-package
• sum_and_multiply
• support-3.8
• support-python-3.10
• switch-str-unicode
• try-update-pypy
• unicode-decode-model-object
• upgrade-packaging-to-setuptools
• upgrade-pre-commit-configuration
• upgrade-tox-ini
• yield-node-in-if

Build # 1688

Build Type

Pull #763

travis-ci

Committed by web-flow

Commit Message

Transform boto3.ServiceRequest to look like dynamic class

`boto3.resource` creates resources dynamically via a resource factory.
Unfortunately that completely breaks static analysis leading to spurious
false positives since pylint cannot determine sanely that attributes
exist or not.

Here's an example of accessing the Topic class out of the `sns` resource.
As you can see, the class is created dynamically rather than existing
in the codebase itself:

```
In [2]: boto3.resource
Out[2]: <function boto3.resource(*args, **kwargs)>

In [3]: boto3.resource('sns')
Out[3]: sns.ServiceResource()

In [4]: boto3.resource('sns').Topic
Out[4]: <bound method ResourceFactory._create_class_partial.<locals>.create_resource of sns.ServiceResource()>
```

This patch adds a fake `__getattr__` method to `ServiceRequest`.
This will prevent `pylint` from emitting `no-member` at all for `ServiceRequest`
instances, but that is a good solution for now until we can load typeshed-like
annotation packages.

Close PyCQA/pylint#3134

Pull Request Pull Request #763: Transform boto3.ServiceRequest to look like dynamic class

Run Details

12 of 12 new or added lines in 1 file covered. (100.0%)

7548 of 8420 relevant lines covered (89.64%)

2.67 hits per line