LIBRARIES-2434 [Analytics Python CLI E2E Testing Tool] #511
Conversation
neelkanth-kaushik
assigned neelkanth-kaushik and MichaelGHSeg and unassigned neelkanth-kaushik
|
|
||
|
|
||
| @click.command() | ||
| @click.option('--writeKey', type=str, help='Segment write key') |
There was a problem hiding this comment.
We'd also like to have an --apihost option
e2e-test/src/cli.py
Outdated
| try: | ||
| # Decode the JSON payload | ||
| decodedJson = json.loads(payload) | ||
| data = json.loads(decodedJson) |
There was a problem hiding this comment.
It looks like this is handling a single event. Could we make the root json datatype an array and send multiple events?
e2e-test/src/cli.py
Outdated
| logger.setLevel(logging.DEBUG) | ||
|
|
||
| # Create a file handler to log messages to a file | ||
| log_directory = f"{os.getenv('LOG_DIRECTORY')}/{datetime.now().strftime('%Y%m%d')}" |
There was a problem hiding this comment.
I don't think we need logging for a simple test, and logs on CI machines are often impossible to get to. If we do want something to be recorded in the build logs for this, we could look into printing to standard error, and reading and reporting from our Golang publisher. I'm not concerned about it at the moment, so just printing errors is fine until we think we need more.
e2e-test/README.md
Outdated
| ### 1. Identify | ||
|
|
||
| ``` bash | ||
| $ e2e-test:run --writeKey='YOUR_WRITE_KEY' --payload='"{\"anonymousId\":\"507f191e810c89729de960ea\",\"channel\":\"browser\",\"context\":{\"ip\":\"8.8.8.8\",\"userAgent\":\"Mozilla/5.0 (Macintosh; Intel Mac OS X 10_9_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/40.0.2214.115 Safari/537.36\"},\"integrations\":{\"All\":false,\"Mixpanel\":true,\"Salesforce\":true},\"messageId\":\"022bb90c-bbac-11e4-8dfc-aa07a5b093q8\",\"traits\":{\"name\":\"Clark Kent\",\"email\":\"[email protected]\",\"plan\":\"premium\",\"logins\":5,\"address\":{\"street\":\"6th St\",\"city\":\"San Francisco\",\"state\":\"CA\",\"postalCode\":\"94103\",\"country\":\"USA\"}},\"type\":\"identify\",\"userId\":\"97980cfea0062\",\"version\":\"2.0\"}"' |
There was a problem hiding this comment.
IPs and userAgents are added by the SDK itself, so we should remove them from the examples.
This commit contains code for Analytics Python CLI E2E Testing Tool - LIBRARIES-2434
Changes in Commit #18827f3e14091da5e7e740a0b0d79cb6cdd07f3c