Updated tests and checks.

Author: isoos

app/lib/frontend/handlers/atom_feed.dart

@@ -9,8 +9,10 @@ import 'package:clock/clock.dart';
 import 'package:crypto/crypto.dart';
 import 'package:shelf/shelf.dart' as shelf;
 
+import '../../admin/actions/actions.dart';
 import '../../package/backend.dart';
 import '../../package/models.dart';
+import '../../package/overrides.dart';
 import '../../shared/configuration.dart';
 import '../../shared/redis_cache.dart';
 import '../../shared/urls.dart' as urls;
@@ -35,6 +37,7 @@ Future<shelf.Response> packageAtomFeedhandler(
   shelf.Request request,
   String package,
 ) async {
+  checkPackageVersionParams(package);
   final feedContent = await cache
       .packageAtomFeedXml(package)
       .get(() => buildPackageAtomFeedContent(package));
@@ -50,19 +53,25 @@ Future<shelf.Response> packageAtomFeedhandler(
 /// Builds the content of the /feed.atom endpoint.
 Future<String> buildAllPackagesAtomFeedContent() async {
   final versions = await packageBackend.latestPackageVersions(limit: 100);
+  versions.removeWhere((pv) => pv.isModerated || pv.isRetracted);
   final feed = _allPackagesFeed(versions);
   return feed.toXmlDocument();
 }
 
 /// Builds the content of the `/packages/<package>/feed.atom` endpoint.
 Future<String> buildPackageAtomFeedContent(String package) async {
+  if (isSoftRemoved(package) ||
+      !await packageBackend.isPackageVisible(package)) {
+    throw NotFoundException.resource(package);
+  }
   final versions = await packageBackend
       .streamVersionsOfPackage(
         package,
         order: '-created',
         limit: 10,
       )
       .toList();
+  versions.removeWhere((pv) => pv.isModerated || pv.isRetracted);
   final feed = _packageFeed(package, versions);
   return feed.toXmlDocument();
 }

app/test/frontend/handlers/invalid_package_url_test.dart

@@ -31,7 +31,7 @@ void main() {
     expect(urls, contains('/packages/<package>/versions/<version>'));
     expect(urls, contains('/api/packages/<package>'));
     // this a naive assertion that fails, if new end-points are introduced!
-    expect(urls, hasLength(43), reason: 'check if new end-points was added');
+    expect(urls, hasLength(44), reason: 'check if new end-points was added');
   });
 
   testWithProfile(