tree de70a427aaea42a9832f708deef50b2a4b4aadea
parent 743261148086abc5cc8f29f1aa1f9b1ed340f5f8
author Nate Bosch <nbosch@google.com> 1612466473 -0800
committer GitHub <noreply@github.com> 1612466473 -0800
gpgsig -----BEGIN PGP SIGNATURE-----
 
 wsBcBAABCAAQBQJgHEkpCRBK7hj4Ov3rIwAAdHIIAImIY+O745X4wrwaqeg73hJZ
 /dJ8AfzKA1GlwLehBZ+xGeaFJGOrW396It0IqVeT8ltBweMUDP13oqS5olXirYR1
 4ZnsYT0hRUYu61hnhlw37MWE88zLR5BOg6BFmmT9J3WPTXiqEMzss21SnB6Poh++
 u4vsuXLBxuWzpXlzUGhpEuLg1zpifsQhwZiwQUkZFJedO/tseRorwTHg7LDM0eWE
 bwhHtTg5sXvqkq5ZJYdzEydSs9IqAwTGS/BoIjkXQ/3UjACf7Twstf9p4uDjnZAN
 eIIdbV6On8SR/DZ8hSeIAW9V2fyFq5e8gGvwMVRNV2sokR3aU8EzTq0OuuYRnH0=
 =Tls2
 -----END PGP SIGNATURE-----
 

Fix header for multispan with null secondary urls (#72)

If a multiple span highlight was created where all the secondary spans
had `null` URLs they were filtered out from the comparison checking
whether multiple files are involved.

This bug was caused during the null safety migration. A similar bug
previously existed if the primary span had a null URL and is also fixed.

- Take a nullable list in `isAllTheSame`. Avoid using `null` as a
  sentinel for no item seen, use `.first` and `.skip(1)` to compare all
  elements.
- Add tests for the case of primary and secondary spans with a null URL.