react-native-opengraph-kit icon indicating copy to clipboard operation
react-native-opengraph-kit copied to clipboard

Automated Test Cases

Open adrienthiery opened this issue 8 years ago • 3 comments

Edge case URLs we are testing :

  • A URL with minified HTML (ex: https://blog.callstack.io/react-native-animations-revisited-part-i-783143d4884)
  • A redirection (ex: http://www.osedea.com)
  • An HTTP Only website (ex: http://www.onlywebsites.com/)
  • A Non latin character website (ex: http://cn.timesofisrael.com/)
  • A Website that allows only google bot through if programmatically called (ex: https://tw.news.yahoo.com/%E3%80%90%E7%8D%A8%E5%AE%B6%E3%80%91%E5%A4%9A%E9%87%8D%E9%98%BB%E5%9A%87%E8%A7%A3%E6%94%BE%E8%BB%8D%E3%80%80%E4%B8%AD%E7%A7%91%E9%99%A2%E5%AF%A6%E6%B8%AC%E5%A4%A9%E5%BC%93%E3%80%81%E5%A4%A9%E5%8A%8D-013928232.html)
  • A URL with a relative image url : (ex: www.centrelapinscretins.com)
  • A URL with 's in the content : (ex: https://blog.womeninmind.com/cKIj-tout-faire-oui-ca-se-peut)

TODO :

  • google.com should provide an image
  • check out Medium articles

If you find any edge case that does not work, please put them here!

adrienthiery avatar Jan 27 '17 16:01 adrienthiery

Possible: https://www.bbc.co.uk/news, where they have a protocol agnostic image tag: <meta property="og:image" content="//m.files.bbci.co.uk/modules/bbc-morph-news-waf-page-meta/1.2.0/bbc_news_logo.png?cb=1">

henrytndrly avatar Jun 16 '17 18:06 henrytndrly

I've found another test case where the ordering of content and property attributes was switched, which caused a problem with an older version of your metaTagOGRegex in OpenGraphParser.js: https://www.inverse.com/article/34343-a-i-scientists-react-to-elon-musk-ai-comments

The latest regex appears to work with this 'format', so just suggesting this as an additional test case.

henrytndrly avatar Jul 19 '17 18:07 henrytndrly

Hi @henrytndrly,

Thanks, I'll add it later :)

adrienthiery avatar Jul 19 '17 18:07 adrienthiery