Fix size check of arrays when broadcasting

[odow]

Closes #295

The previous check for length is incorrect because it ignores length 1 dimensions. But the comment explains some arrays do not support size. Which? SparseAxisArray? Do we really support broadcasting over that?

[codecov]

Codecov Report

Attention: Patch coverage is 72.72727% with 3 lines in your changes missing coverage. Please review.

Project coverage is 91.74%. Comparing base (0e42a16) to head (d5c3985).

Files	Patch %	Lines
src/broadcast.jl	72.72%	3 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #296      +/-   ##
==========================================
- Coverage   91.85%   91.74%   -0.11%     
==========================================
  Files          23       23              
  Lines        2209     2216       +7     
==========================================
+ Hits         2029     2033       +4     
- Misses        180      183       +3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[blegat]

This is an issue you detected with the integration script: #160
Maybe we should check which one it was and update the comment

[odow]

Oooo. That's a bad bug for me to introduce.

It was SparseAxisArray because when I had size the JuMP tests failed.