o erro seria
messaget := testcursor.column1; já que o cursor está fechado (você deve apenas usar testcursorrec.column2 . seu código não está verificando nenhuma linha, nem linhas duplicadas. você pode simplificar isso para
create or replace function testfunction
(
somevalue in table1.column1%type
)
return table1.column2%type
AS
messaget table1.column2%type; -- use %type where possible.
begin
select t.column2
into messaget
from table1 t
where t.column1 = somevalue
and rownum = 1;--only if you dont care if theres 2+ rows.
return messaget;
exception
when no_data_found
then
return null; -- if you want to ignore no rows.
end;
