Skip to content
Snippets Groups Projects

Patterns corrected for condenser/follow/tags_api groups (both old and tavern)

Merged Andrzej Lisak requested to merge tavern_pattern_fixes into master
1 unresolved thread

Corrections include:

  • rebranding
  • accepted different future cashout_time (prior to HF17 it was taken from root and we don't want to introduce hardfork differences to HiveMind)
  • accepted different post_id
  • accepted lack of double-escaped slashes in json_metadata (now is better)
  • corrected indent for tavern's 2 spaces
  • corrected cases of CLRF

Comparison script now removes .diff.json and .out.json files from previous run. It also generates .out.json with error message in case of error/no response (in other words all failing tests have .out.json file).

Added comments to failing tests why they are failing and to tests with empty results why there is no other option.

Some tests were marked as passing - that is valid for hivemind!61 (merged)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
6 6 "beneficiaries": [],
7 7 "body": "Ty jeste\u015b autorem tego tekstu?\nhttps://web.facebook.com/notes/szymon-szewczyk-blog/nasienie-wolno%C5%9Bci/1062629210473658",
8 8 "body_length": 122,
9 "cashout_time": "2016-09-18T21:19:45",
9 "cashout_time": "1990-01-01T00:00:00",
  • Just out of curiosity, is this just some default value for things that don't have a valid cashout time? I'm asking because 1990 was before blockchain started.

    Edited by Dan Notestein
  • Author Maintainer

    At times value displayed as cashout_time is taken from hive_posts.payout_at field and its default value is 1990-01-01 for some reason (tbh. I only spotted it is indeed strange when you pointed that out - shouldn't it rather be 1970-01-01 like in other fields where date is stored?)

  • I spoke to @jsalyers just now, and he confirms that hivemind does default to 1990 instead of 1970 (but he doesn't know why).

  • Apparently it's defined that way in schema file, but still no idea why. Anyways, it looks fine, so going to merge this in.

  • Please register or sign in to reply
  • Dan Notestein approved this merge request

    approved this merge request

  • merged

  • Dan Notestein mentioned in commit 886dc5c0

    mentioned in commit 886dc5c0

  • Please register or sign in to reply
    Loading