Skip to content

Conversation

@jacalata
Copy link
Contributor

Trying to get a repr implemented on all models. While checking on this I realized that we should have code coverage working - found a typo in the action.

fix: new test found that group_item says name is optional in init, but not optional as a property. removed the non_nullable checks.

@jacalatajacalata changed the base branch from master to developmentSeptember 23, 2023 07:37
@github-actions
Copy link

Coverage

Coverage Report
FileStmtsMissCoverMissing
tableauserverclient
__init__.py40100%
_version.py279163163 42%
config.py50100%
datetime_helpers.py2511 96%
exponential_backoff.py200100%
filesys_helpers.py310100%
namespace.py2633 88%
tableauserverclient/helpers
__init__.py10100%
logging.py20100%
strings.py2711 96%
tableauserverclient/models
__init__.py360100%
column_item.py553232 42%
connection_credentials.py351111 69%
connection_item.py771010 87%
custom_view_item.py11999 92%
data_acceleration_report_item.py5411 98%
data_alert_item.py15933 98%
database_item.py2033434 83%
datasource_item.py2401111 95%
dqw_item.py10455 95%
exceptions.py40100%
favorites_item.py5788 86%
fileupload_item.py190100%
flow_item.py1491010 93%
flow_run_item.py720100%
group_item.py8133 96%
interval_item.py1471313 91%
job_item.py1771212 93%
metric_item.py1301212 91%
pagination_item.py340100%
permissions_item.py7577 91%
project_item.py1473131 79%
property_decorators.py961818 81%
reference_item.py190100%
revision_item.py6033 95%
schedule_item.py19466 97%
server_info_item.py3955 87%
site_item.py6101212 98%
subscription_item.py10333 97%
table_item.py1181818 85%
tableau_auth.py712626 63%
tableau_types.py2311 96%
tag_item.py160100%
target.py60100%
task_item.py5611 98%
user_item.py2771717 94%
view_item.py1561515 90%
webhook_item.py660100%
workbook_item.py2551616 94%
tableauserverclient/server
__init__.py80100%
exceptions.py40100%
filter.py2111 95%
pager.py3744 89%
query.py1092121 81%
request_factory.py899135135 85%
request_options.py19744 98%
server.py1631717 90%
sort.py60100%
tableauserverclient/server/endpoint
__init__.py270100%
auth_endpoint.py761212 84%
custom_views_endpoint.py6822 97%
data_acceleration_report_endpoint.py210100%
data_alert_endpoint.py972525 74%
databases_endpoint.py1033333 68%
datasources_endpoint.py3044141 87%
default_permissions_endpoint.py4655 89%
dqw_endpoint.py431616 63%
endpoint.py1993333 83%
exceptions.py6066 90%
favorites_endpoint.py952222 77%
fileuploads_endpoint.py510100%
flow_runs_endpoint.py631010 84%
flows_endpoint.py2136060 72%
groups_endpoint.py1071111 90%
jobs_endpoint.py5888 86%
metadata_endpoint.py881414 84%
metrics_endpoint.py5888 86%
permissions_endpoint.py4655 89%
projects_endpoint.py1272222 83%
resource_tagger.py401010 75%
schedules_endpoint.py1001111 89%
server_info_endpoint.py351010 71%
sites_endpoint.py1262828 78%
subscriptions_endpoint.py581515 74%
tables_endpoint.py1023939 62%
tasks_endpoint.py6266 90%
users_endpoint.py1271515 88%
views_endpoint.py1241010 92%
webhooks_endpoint.py571111 81%
workbooks_endpoint.py3284242 87%
TOTAL9312123387%

@jacalatajacalata merged commit 2f6a343 into developmentSep 25, 2023
@jacalatajacalata deleted the jac/repr_coverage branch September 25, 2023 23:17
jacalata added a commit that referenced this pull request Oct 4, 2023
* Code coverage and pretty printing (#1283) * implement str and repr for a bunch more classes * also: JWT, user-impersonation (cherry picked from commit 4887a62) * fix code coverage action * use reflection to find all models for comprehensive testing. --------- Co-authored-by: Lee Graber <[email protected]> * update publish action (#1286) * 0.27 (#1272) * Bump urllib3 from 2.0.4 to 2.0.6 (#1287) * Bump urllib3 from 2.0.4 to 2.0.6 Bumps [urllib3](https://github.com/urllib3/urllib3) from 2.0.4 to 2.0.6. - [Release notes](https://github.com/urllib3/urllib3/releases) - [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst) - [Commits](urllib3/urllib3@2.0.4...2.0.6) --- updated-dependencies: - dependency-name: urllib3 dependency-type: direct:production ... Signed-off-by: dependabot[bot] <[email protected]> --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Lee Graber <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@jacalata@bcantoni@LGraber