Why isn't there a throwing else block, would that also make the tests more complicated? It seems wrong to me to accept package uploads without a package ID in the package (although it's likely we would fail before we got to this point, if that is the case maybe we don't need an if statement?).