MarketSquare / MarketSquare/robotframework-faker

Review data provider arguments.

Open
#88 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
HTML
Stars
69
Forks
23
PR merge metrics
No merged PRs in 30d

Description

There are a few keywords which from the docs display arguments as `*args, **kwargs` +. Beside being unclear on what these arguments are, I am not sure the current implementation can properly translate data types (if any are required). So we should review arguments of the data providers and see if we can better support passing arguments from robot to faker on data providers.

\+ _Footnote: The fact that arguments are unnamed or generic itself might only be solved within Faker. There is some options to correct that within the FakerLibrary side. But the maintenance and change cost may be too high for that._

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

The issue names no files or tests; start by locating the data provider keywords and reviewing the documented `*args, **kwargs` signatures. Compare how arguments and required data types pass from Robot Framework to Faker, with completion defined as clearer argument support or a documented decision about the remaining Faker limitation.

Written by the indexing model from the issue text.

Assessment

Domain
tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.