Skip to content

Commit

Permalink
feat: send domInfoSha when posting comparisons
Browse files Browse the repository at this point in the history
  • Loading branch information
nilshah98 committed May 25, 2023
1 parent d365dc3 commit 2591a74
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 2 deletions.
5 changes: 3 additions & 2 deletions packages/client/src/client.js
Original file line number Diff line number Diff line change
Expand Up @@ -366,7 +366,7 @@ export class PercyClient {
return snapshot;
}

async createComparison(snapshotId, { tag, tiles = [], externalDebugUrl, ignoredElementsData } = {}) {
async createComparison(snapshotId, { tag, tiles = [], externalDebugUrl, domInfoSha, ignoredElementsData } = {}) {
validateId('snapshot', snapshotId);

this.log.debug(`Creating comparision: ${tag.name}...`);
Expand All @@ -386,7 +386,8 @@ export class PercyClient {
type: 'comparisons',
attributes: {
'external-debug-url': externalDebugUrl || null,
'ignore-elements-data': ignoredElementsData || null
'ignore-elements-data': ignoredElementsData || null,
'dom-info-sha': domInfoSha || null,
},
relationships: {
tag: {
Expand Down
3 changes: 3 additions & 0 deletions packages/webdriver-utils/src/providers/genericProvider.js
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,10 @@ export default class GenericProvider {
tag,
tiles,
// TODO: Fetch this one for bs automate, check appium sdk
// TODO: Need to update external debug url validation in api
externalDebugUrl: this.debugUrl,
// TODO: pass actual dom info sha
domInfoSha: null,
environmentInfo: ENV_INFO,
clientInfo: CLIENT_INFO
};
Expand Down

0 comments on commit 2591a74

Please sign in to comment.