The synopsis for Exporter says:

package YourModule;
require Exporter;
@ISA = qw(Exporter);

You omitted the 'require Exporter'.

I prefer this version personally:

package one;
use strict;
use warnings;
use base qw(Exporter);
our @EXPORT = qw(method1);

sub method1 {
     print "method1\n";
}

1;
[download]